[25.0 backport] vendor: golang.org/x/sys v0.18.0, golang.org/x/term v0.18.0, golang.org/x/crypto v0.21.0, golang.org/x/net v0.23.0 #5279
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
full diff: golang/sys@v0.16.0...v0.18.0 Signed-off-by: Sebastiaan van Stijn <github@gone.nl> (cherry picked from commit 9a2133f) Signed-off-by: Austin Vazquez <macedonv@amazon.com>
no changes in vendored code full diff: golang/term@v0.15.0...v0.18.0 Signed-off-by: Sebastiaan van Stijn <github@gone.nl> (cherry picked from commit c7a50eb) Signed-off-by: Austin Vazquez <macedonv@amazon.com>
full diffs changes relevant to vendored code: - golang/net@v0.19.0...v0.22.0 - http2: remove suspicious uint32->v conversion in frame code - http2: send an error of FLOW_CONTROL_ERROR when exceed the maximum octets - golang/crypto@v0.17.0...v0.21.0 - (no changes in vendored code) Signed-off-by: Sebastiaan van Stijn <github@gone.nl> (cherry picked from commit 4745b95) Signed-off-by: Austin Vazquez <macedonv@amazon.com>
full diff: golang/net@v0.22.0...v0.23.0 Includes a fix for CVE-2023-45288, which is also addressed in go1.22.2 and go1.21.9; > http2: close connections when receiving too many headers > > Maintaining HPACK state requires that we parse and process > all HEADERS and CONTINUATION frames on a connection. > When a request's headers exceed MaxHeaderBytes, we don't > allocate memory to store the excess headers but we do > parse them. This permits an attacker to cause an HTTP/2 > endpoint to read arbitrary amounts of data, all associated > with a request which is going to be rejected. > > Set a limit on the amount of excess header frames we > will process before closing a connection. > > Thanks to Bartek Nowotarski for reporting this issue. Signed-off-by: Sebastiaan van Stijn <github@gone.nl> (cherry picked from commit 5fcbbde) Signed-off-by: Austin Vazquez <macedonv@amazon.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## 25.0 #5279 +/- ##
==========================================
- Coverage 61.36% 61.35% -0.01%
==========================================
Files 290 290
Lines 20098 20098
==========================================
- Hits 12334 12332 -2
- Misses 6871 6872 +1
- Partials 893 894 +1 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
- What I did
Backports #4998 to 25.0
- How I did it
- How to verify it
CI must be successful
- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)