Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to JRuby 9.4.3.0 #1234

Merged
merged 2 commits into from
Sep 11, 2023
Merged

Update to JRuby 9.4.3.0 #1234

merged 2 commits into from
Sep 11, 2023

Conversation

headius
Copy link
Contributor

@headius headius commented Sep 11, 2023

JRuby 9.4.3.0 includes an updated Psych YAML library, which uses SnakeYAML-Engine and avoids several CVEs against the original SnakeYAML. By updating here, downstream users of asciidoctorj will not run into security audit issues.

See related issues and PRs:

Thank you for opening a pull request and contributing to AsciidoctorJ!

Please take a bit of time giving some details about your pull request:

Kind of change

  • Bug fix (sorta, update JRuby to get away from false-ish CVEs in SnakeYAML)
  • New non-breaking feature
  • New breaking feature
  • Documentation update
  • Build improvement

Description

What is the goal of this pull request?

  • Update JRuby to get a version that does not use CVE-ridden SnakeYAML

How does it achieve that?

  • Updates JRuby to 9.4.3.0

Are there any alternative ways to implement this?

  • Not really

Are there any implications of this pull request? Anything a user must know?

  • YAML compliance rises to 1.2, which may reject YAML that was valid in 1.0 or 1.1.

Release notes

Please add a corresponding entry to the file CHANGELOG.adoc

JRuby 9.4.3.0 includes an udpated Psych YAML library, which uses
SnakeYAML-Engine and avoids several CVEs against the original
SnakeYAML. By updating here, downstream users of asciidoctorj will
not run into security audit issues.

See related issues and PRs:

* jruby/jruby#7570
* jruby/jruby#7600
* jruby/jruby#7626
* jruby/jruby#7935
@headius
Copy link
Contributor Author

headius commented Sep 11, 2023

Ok I misspoke... current asciidoctorj uses 9.4.1.0 which DOES use the newer Psych that switched to SnakeYAML-Engine. So there's no immediate rush on merging and releasing this PR.

Sorry for the noise. You can close this if you like, but of course 9.4.3.0 is better than 9.4.1.0 (and 9.4.4.0 will be better than that in a week or two).

@headius
Copy link
Contributor Author

headius commented Sep 11, 2023

FYI I deleted a comment here that was intended for another project.

@robertpanzer robertpanzer merged commit 9f2d1ff into asciidoctor:main Sep 11, 2023
@robertpanzer
Copy link
Member

Thank you so much!
Really appreciate this.

@headius headius deleted the jruby_update branch September 11, 2023 21:03
@robertpanzer robertpanzer mentioned this pull request Sep 17, 2023
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants