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

Fix http parsing of repeated headers #6325

Merged
merged 1 commit into from
Feb 8, 2018
Merged

Conversation

adriansr
Copy link
Contributor

@adriansr adriansr commented Feb 8, 2018

Packetbeat HTTP protocol parser was not concatenating properly repeated
headers in a request or response. This caused corruption in the header
in the form of null bytes (\u0000).

@adriansr adriansr added bug review Packetbeat needs_backport PR is waiting to be backported to other branches. labels Feb 8, 2018
Packetbeat HTTP protocol parser was not concatenating properly repeated
headers in a request or response. This caused corruption in the header
in the form of null bytes (\u0000).
Copy link
Contributor

@tsg tsg left a comment

Choose a reason for hiding this comment

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

LGTM 🎉

@tsg tsg merged commit eff26f2 into elastic:master Feb 8, 2018
@tsg tsg added the v6.2.2 label Feb 8, 2018
@tsg
Copy link
Contributor

tsg commented Feb 8, 2018

Can you backport to the 6.2 branch, please?

@adriansr
Copy link
Contributor Author

adriansr commented Feb 8, 2018

@tsg here #6330

adriansr added a commit to adriansr/beats that referenced this pull request Apr 6, 2018
Packetbeat HTTP protocol parser was not concatenating properly repeated
headers in a request or response. This caused corruption in the header
in the form of null bytes (\u0000).
andrewkroh pushed a commit that referenced this pull request Apr 6, 2018
Packetbeat HTTP protocol parser was not concatenating properly repeated
headers in a request or response. This caused corruption in the header
in the form of null bytes (\u0000).
@adriansr adriansr removed the needs_backport PR is waiting to be backported to other branches. label Jun 15, 2018
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.

2 participants