π¨ [security] [src/api - 2.10] Update rack 2.2.22 β 2.2.23 (patch)#19521
Open
depfu[bot] wants to merge 3 commits into2.10from
Open
π¨ [security] [src/api - 2.10] Update rack 2.2.22 β 2.2.23 (patch)#19521depfu[bot] wants to merge 3 commits into2.10from
depfu[bot] wants to merge 3 commits into2.10from
Conversation
00f13db to
4974baa
Compare
Member
|
@depfu rebase |
This file contains hidden or 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
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.
π¨ Your current dependencies have known security vulnerabilities π¨
This dependency update fixes known security vulnerabilities. Please see the details below and assess their impact carefully. We recommend to merge and deploy this as soon as possible!
Here is everything you need to know about this update. Please take a good look at what changed and the test results before merging this pull request.
What changed?
Security Advisories π¨
π¨ Rack::Static prefix matching can expose unintended files under the static root
π¨ Rack:: Static header_rules bypass via URL-encoded paths
π¨ Rack's multipart byte range processing allows denial of service via excessive overlapping ranges
π¨ Rack's greedy multipart boundary parsing can cause parser differentials and WAF bypass.
π¨ Rack has quadratic complexity in Rack::Utils.select_best_encoding via wildcard Accept-Encoding header
π¨ Rack has a root directory disclosure via unescaped regex interpolation in Rack::Directory
π¨ Rack's multipart parsing without Content-Length header allows unbounded chunked file uploads
π¨ Rack::Sendfile header-based X-Accel-Mapping regex injection enables unauthorized X-Accel-Redirect
π¨ Rack has Content-Length mismatch in Rack::Files error responses
Release Notes
2.2.23 (from changelog)
Does any of this look wrong? Please let us know.
Commits
See the full diff on Github. The new version differs by 19 commits:
Bump patch version.Fix tests for old Rubies.Add version guard around non-default gems.Fix handling of `Errno::EPIPE` in multipart tests.Fix typo in test.Add `ostruct` to Gemfile.Fix test expectation.Add `logger` to gemfile.Add Ruby v4.0 to the test matrix.Add Content-Length size check in Rack::Multipart::ParserFix root prefix bug in Rack::StaticOnly do a simple substitution on the x-accel-mapping pathsUse a default limit of 100 byte rangesUse `String#bytesize` for `Content-Length` in error responses.Fix `header_rules` bypass via URL-encoded paths.Raise error for multipart requests with multiple boundary parametersAvoid O(n^2) algorithm in Rack::Utils.select_best_encodingRoot directory disclosure via unescaped regex interpolation in `Rack::Directory`.Add missing releases to changelog.Depfu will automatically keep this PR conflict-free, as long as you don't add any commits to this branch yourself. You can also trigger a rebase manually by commenting with
@depfu rebase.All Depfu comment commands