Skip to content

Conversation

@jsvd
Copy link
Contributor

@jsvd jsvd commented Apr 30, 2024

Beware that 'mvn package' will create jar with the current maven version being used, so it is necessary to use maven 3.9.6 while running the packaging task.

I set the gem version to 3.9.6 since this gem has been versioned according to the version of maven it packages.

Beware that 'mvn package' will create jar with the current maven version being used,
so it is necessary to use maven 3.9.6 while running the packaging task.
@enebo enebo merged commit dbda6ef into jruby:master Apr 30, 2024
@jsvd
Copy link
Contributor Author

jsvd commented May 2, 2024

Thank you @enebo, any chance we could get this released? Please remember the maven being used to run mvn package is the one that ends up in the gem, from my experiments, so you have to use 3.9.6.
The reason is to avoid having some guava related cves in our JRuby project.

@jsvd jsvd deleted the upgrade_maven_3_9_6 branch May 2, 2024 09:01
@headius
Copy link
Member

headius commented May 2, 2024

Sorry I forgot about releasing these updates in the flurry of activity over the past few months.

@headius
Copy link
Member

headius commented May 2, 2024

Sorry I forgot about releasing these updates in the flurry of activity over the past few months.

That is to say that I will look into releases today! 😀

@headius
Copy link
Member

headius commented May 2, 2024

ruby-maven-libs 3.9.6 has been pushed to rubygems.org!

@jsvd
Copy link
Contributor Author

jsvd commented May 2, 2024

@headius Unfortunately it seems your local dir still had previous jars:

❯ gem unpack ruby-maven-libs-3.9.6.gem
Unpacked gem: '/private/tmp/ruby-maven-libs-3.9.6'

/tmp
❯ find ruby-maven-libs-3.9.6 -name "*maven-core*"
ruby-maven-libs-3.9.6/maven-home/lib/maven-core-3.8.7.jar
ruby-maven-libs-3.9.6/maven-home/lib/maven-core-3.9.6.jar

Maybe creating a 3.9.6.1 ? 😅

@headius
Copy link
Member

headius commented May 2, 2024

Blast it all, I didn't realize there were unclean remnants or that they'd get included.

I see the correct jar is there, but I assume since the bad jar is there it still gets flagged for CVEs?

@headius
Copy link
Member

headius commented May 2, 2024

I'm going to push a 3.9.6.1.pre1 and you can confirm it looks like it should.

@jsvd
Copy link
Contributor Author

jsvd commented May 2, 2024

I see the correct jar is there, but I assume since the bad jar is there it still gets flagged for CVEs?

that is right 😞

I'm going to push a 3.9.6.1.pre1 and you can confirm it looks like it should.

ready when you are

@headius
Copy link
Member

headius commented May 2, 2024

@headius
Copy link
Member

headius commented May 2, 2024

I also pushed a commit that updates the clean target to also clean out maven-home. 😡

@jsvd
Copy link
Contributor Author

jsvd commented May 2, 2024

LGTM, I can confirm the files are not duplicated any more and that scanners are clean for that prereleased version. 👌

@headius
Copy link
Member

headius commented May 2, 2024

Can you confirm for me that it still works properly? The mismatched Maven version and Gem version scare me a little bit.

@jsvd
Copy link
Contributor Author

jsvd commented May 2, 2024

Given that the 3.9.6 version is still here:

/tmp/ruby-maven-libs-3.9.6.1.pre1
❯ grep VERSION lib/maven.rb 
  VERSION = '3.9.6'.freeze

It should be fine, but testing now, will report in a few min.

@headius
Copy link
Member

headius commented May 2, 2024

Given that the 3.9.6 version is still here

Yeah I'm hopeful that's good enough, but I had to hack the build to not use that as the gem version so I just need confirmation.

@jsvd
Copy link
Contributor Author

jsvd commented May 2, 2024

Worked correctly! I was able to use a Logstash version with jruby 9.4.7.0 w/ ruby-maven-libs 3.9.6.1.pre1 and install a Logstash plugin that downloads and installs jars at install time 👌 LGTM

@jsvd
Copy link
Contributor Author

jsvd commented May 2, 2024

Also:

❯ grype  -q ruby-maven-libs-3.8.9
NAME   INSTALLED     FIXED-IN        TYPE          VULNERABILITY        SEVERITY 
guava  25.1-android  32.0.0-android  java-archive  GHSA-7g45-4rm6-3mm3  Medium    
guava  25.1-android  32.0.0-android  java-archive  GHSA-5mg8-w23w-74h3  Low

❯ grype  -q ruby-maven-libs-3.9.6
NAME   INSTALLED     FIXED-IN        TYPE          VULNERABILITY        SEVERITY 
guava  25.1-android  32.0.0-android  java-archive  GHSA-7g45-4rm6-3mm3  Medium    
guava  25.1-android  32.0.0-android  java-archive  GHSA-5mg8-w23w-74h3  Low

❯ grype  -q ruby-maven-libs-3.9.6.1.pre1
No vulnerabilities found

headius added a commit that referenced this pull request May 2, 2024
This release includes Maven 3.9.6 and is only published because
the gem version 3.9.6 accidentally contained 3.8.7 jars.

See ##5
@headius
Copy link
Member

headius commented May 2, 2024

Thanks for the confirmation @jsvd! I've pushed 3.9.6.1 to rg.org.

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.

3 participants