[balsa] Disallow CR not followed by LF in header values. #34829
Merged
CI (Envoy) / Mobile/Android
skipped
Jun 25, 2024 in 0s
Check was skipped
This check was not triggered in this CI run
Details
Request (pr/34829/main@f15b9f2)
@bencebeky f15b9f2
#34829 merge
main@128a99a
[balsa] Disallow CR not followed by LF in header values.
Commit Message: [balsa] Disallow CR not followed by LF in header values.
Additional Description: This makes the behavior of BalsaParser consistent with that of http-parser.
Risk Level: low
Testing: //test/common/http/http1:codec_impl_test
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: n/a
Environment
Request variables
Key | Value |
---|---|
ref | bb0d0c5 |
sha | f15b9f2 |
pr | 34829 |
base-sha | 128a99a |
actor | @bencebeky |
message | [balsa] Disallow CR not followed by LF in header values.... |
started | 1719328486.988361 |
target-branch | main |
trusted | false |
Build image
Container image/s (as used in this CI run)
Key | Value |
---|---|
default | envoyproxy/envoy-build-ubuntu:f94a38f62220a2b017878b790b6ea98a0f6c5f9c |
mobile | envoyproxy/envoy-build-ubuntu:mobile-f94a38f62220a2b017878b790b6ea98a0f6c5f9c |
Version
Envoy version (as used in this CI run)
Key | Value |
---|---|
major | 1 |
minor | 31 |
patch | 0 |
dev | true |
Loading