Skip to content

Library updates for cve's, suppression cleanup and not fail github action job SonarCloud if SONARCLOUD_TOKEN not found (summary report instead).#1667

Open
duttonw wants to merge 2 commits into
apache:mainfrom
duttonw:main

Conversation

@duttonw

@duttonw duttonw commented Apr 21, 2026

Copy link
Copy Markdown

Library updates:

  • profile jakartaee11: jakarta-ee from 11.0.0-M5 to 11.0.0
  • Spring Version from 6.2.12 to 7.0.7 (we are 2 months off OSS EOL for 6.2.x)
  • struts2-jasperreports-plugin: jasperreports from 6.21.3 to 6.21.5
  • struts2-jasperreports7-plugin: jasperreports from 7.0.4 to 7.0.6
  • profile jakartaee11: jakarta-ee from 11.0.0-M5 to 11.0.0

Dependancy suppression cleanup

Sonar github workflow to be skipped if secret is not found


Struts & Spring Compatibility Matrix

Struts Plugin Version Compatible Spring Version Spring OSS End of Life (EOL)
7.1.1 6.2.x 2026-06
7.1.1 7.0.7 (Jakarta EE 11 only) 2027-06

@lukaszlenart lukaszlenart left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would wait with switching to Spring 7.x and Jakarta 11 - this is a major upgrade and it must be properly communicated, we need a JIRA ticket for that.

Comment thread pom.xml Outdated
Comment thread pom.xml
@lukaszlenart

Copy link
Copy Markdown
Member

Merge conflicts

@duttonw
duttonw force-pushed the main branch 7 times, most recently from 12cbad7 to f75e89e Compare July 21, 2026 05:44
* Library updates
* Dependancy suppression cleanup
@duttonw

duttonw commented Jul 21, 2026

Copy link
Copy Markdown
Author

@lukaszlenart rebased and put back to be spring6 base. Also fixed up owasp testing results (including caching of db). Enabled test results to be surfaced by github actions as well as annotations.

@lukaszlenart lukaszlenart left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All looks good, let me know if you are going to address this few non-blocking issues? I'm fine to address them later

push:
branches:
- 'main'
- 'develop'

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We do not use the "develop" branch, but that's fine

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is more for forks

Comment thread pom.xml Outdated
Comment thread .github/workflows/maven.yml
…se use mirror for forks not configured, remove workflow_call for now since we don't on call
@duttonw

duttonw commented Jul 21, 2026

Copy link
Copy Markdown
Author

Updated per your comments, I've allowed NIST_NVD_API_KEY usage as well as non (do note that NIST make take a very long time to load cache so lets see how we go on this utilisation, I've seen up to 6hours to re-create the cache db)
https://github.com/duttonw/struts/actions/runs/29867535204/job/88759383394

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