Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated dependency github.com/tsg/gopacket #6583

Merged
merged 2 commits into from
Mar 19, 2018

Conversation

adriansr
Copy link
Contributor

@adriansr adriansr commented Mar 16, 2018

Addresses packetbeat termination problems with both af_packet and pcap captures.

Fixes #6535

Fixes packetbeat termination problems with both af_packet and pcap
captures.

Fixes elastic#6536
@adriansr adriansr force-pushed the vendor/update_tsg_gopacket branch from 3f96d8d to ea94cc3 Compare March 17, 2018 17:31
@ruflin
Copy link
Contributor

ruflin commented Mar 19, 2018

Could you add a CHANGELOG entry?

@ruflin
Copy link
Contributor

ruflin commented Mar 19, 2018

@tsg Makes we wonder if we should move gopacket under the elastic repo?

Copy link
Member

@andrewkroh andrewkroh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This needs a changelog entry mentioning it fixes the issue in #6535.

@adriansr
Copy link
Contributor Author

Fyi I referenced the wrong issue, it is #6535.

Adding it to CHANGELOG now

@andrewkroh andrewkroh merged commit 2f4849c into elastic:master Mar 19, 2018
adriansr added a commit to adriansr/beats that referenced this pull request Apr 5, 2018
Fixes packetbeat termination problems with both af_packet and pcap
captures.

Fixes elastic#6535
andrewkroh pushed a commit that referenced this pull request Apr 6, 2018
Fixes packetbeat termination problems with both af_packet and pcap
captures.

Fixes #6535
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
Fixes packetbeat termination problems with both af_packet and pcap
captures.

Fixes elastic#6535
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants