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

Next/20200909/v3 #5396

Closed
wants to merge 10 commits into from
Closed

Conversation

victorjulien
Copy link
Member

#5390 #5391 #5392

PRScript output (if applicable):
Passed

catenacyber and others added 10 commits September 9, 2020 12:31
To signal incomplete data, we must return the number of
consumed bytes. When we get a banner and some records, we have
to take into account the number of bytes already consumed by
the banner parsing before reaching an incomplete record.
This parameter is NULL or the pointer to the previous state
for the previous protocol in the case of a protocol change,
for instance from HTTP1 to HTTP2

This way, the new protocol can use the old protocol context.
For instance, HTTP2 mimicks the HTTP1 request, to have a HTTP2
transaction with both request and response
moved to suricata-verify
ie do not stop on first try if we do not have enough data
@victorjulien victorjulien requested review from jasonish and a team as code owners September 9, 2020 14:21
@catenacyber catenacyber mentioned this pull request Sep 9, 2020
@victorjulien
Copy link
Member Author

Merged with a commit msg update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants