Skip to content

🚨 [security] Update ruby-saml 1.17.0 → 1.18.0 (minor)#1437

Closed
depfu[bot] wants to merge 1 commit intomainfrom
depfu/update/ruby-saml-1.18.0
Closed

🚨 [security] Update ruby-saml 1.17.0 → 1.18.0 (minor)#1437
depfu[bot] wants to merge 1 commit intomainfrom
depfu/update/ruby-saml-1.18.0

Conversation

@depfu
Copy link
Contributor

@depfu depfu bot commented Mar 12, 2025


🚨 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?

↗️ ruby-saml (indirect, 1.17.0 → 1.18.0) · Repo · Changelog

Security Advisories 🚨

🚨 Ruby SAML allows a SAML authentication bypass due to namespace handling (parser differential)

Summary

An authentication bypass vulnerability was found in ruby-saml due to a parser differential.
ReXML and Nokogiri parse XML differently, the parsers can generate entirely different document structures from the same XML input. That allows an attacker to be able to execute a Signature Wrapping attack.

Impact

This issue may lead to authentication bypass.

🚨 Ruby SAML allows a SAML authentication bypass due to DOCTYPE handling (parser differential)

Summary

An authentication bypass vulnerability was found in ruby-saml due to a parser differential.
ReXML and Nokogiri parse XML differently, the parsers can generate entirely different document structures from the same XML input. That allows an attacker to be able to execute a Signature Wrapping attack.

Impact

This issue may lead to authentication bypass.

🚨 Ruby SAML allows remote Denial of Service (DoS) with compressed SAML responses

Summary

ruby-saml is susceptible to remote Denial of Service (DoS) with compressed SAML responses.

Ruby-saml uses zlib to decompress SAML responses in case they're compressed. It is possible to bypass the message size check with a compressed assertion since the message size is checked before inflation and not after.

Impact

This issue may lead to remote Denial of Service (DoS).

Commits

See the full diff on Github. The new version differs by 31 commits:


Depfu Status

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
@​depfu rebase
Rebases against your default branch and redoes this update
@​depfu recreate
Recreates this PR, overwriting any edits that you've made to it
@​depfu merge
Merges this PR once your tests are passing and conflicts are resolved
@​depfu cancel merge
Cancels automatic merging of this PR
@​depfu close
Closes this PR and deletes the branch
@​depfu reopen
Restores the branch and reopens this PR (if it's closed)
@​depfu pause
Ignores all future updates for this dependency and closes this PR
@​depfu pause [minor|major]
Ignores all future minor/major updates for this dependency and closes this PR
@​depfu resume
Future versions of this dependency will create PRs again (leaves this PR as is)

@depfu depfu bot added the depfu label Mar 12, 2025
@mitlib mitlib temporarily deployed to thesis-submit-pr-1437 March 12, 2025 22:09 Inactive
@coveralls
Copy link

Coverage Status

coverage: 98.313%. remained the same
when pulling 57607a0 on depfu/update/ruby-saml-1.18.0
into 757e20a on main.

@depfu
Copy link
Contributor Author

depfu bot commented Mar 13, 2025

Closing because this update has already been applied

@depfu depfu bot closed this Mar 13, 2025
@depfu depfu bot deleted the depfu/update/ruby-saml-1.18.0 branch March 13, 2025 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants