Skip to content

[9.1.x] Backport HTTP Validations#9016

Merged
bryancall merged 5 commits intoapache:9.1.xfrom
masaori335:asf-9.1.x-0809
Aug 9, 2022
Merged

[9.1.x] Backport HTTP Validations#9016
bryancall merged 5 commits intoapache:9.1.xfrom
masaori335:asf-9.1.x-0809

Conversation

@masaori335
Copy link
Contributor

@masaori335 masaori335 commented Aug 9, 2022

Backport #9009, #9010, #9011, #9012, and #9013 to the 9.1.x branch.

@masaori335 masaori335 added the Backport Marked for backport for an LTS patch release label Aug 9, 2022
@masaori335 masaori335 requested a review from bryancall August 9, 2022 01:44
@masaori335 masaori335 requested a review from zwoop as a code owner August 9, 2022 01:44
@masaori335 masaori335 self-assigned this Aug 9, 2022
@masaori335 masaori335 added this to the 9.1.3 milestone Aug 9, 2022
@masaori335
Copy link
Contributor Author

[approve ci fedora]

maskit and others added 5 commits August 9, 2022 11:08
(cherry picked from commit eaef5e8)

Conflicts:
	proxy/http2/HTTP2.cc
Strictly following RFC 3986 Section 3.1

```
scheme      = ALPHA *( ALPHA / DIGIT / "+" / "-" / "." )
```

(cherry picked from commit c56f872)

Conflicts:
	proxy/http2/HTTP2.cc
(cherry picked from commit 2f363d9)

Conflicts:
	tests/gold_tests/headers/good_request_after_bad.test.py
	tests/gold_tests/logging/gold/field-json-test.gold
	tests/gold_tests/logging/log-field-json.test.py
…NUATION frame (apache#9012)

* Add content length mismatch check on handling HEADERS frame and CONTINUATION frame

* Correct error class of HTTP/2 malformed requests

(cherry picked from commit e921228)
@bryancall bryancall merged commit 2e3a50c into apache:9.1.x Aug 9, 2022
Copy link
Contributor

@bryancall bryancall left a comment

Choose a reason for hiding this comment

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

👍🏻

@bryancall bryancall changed the title [9.1.x] Backport HTTP Validations [9.1.x] Backport HTTP Validations Aug 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backport Marked for backport for an LTS patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants