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

Use tomcat without critical errors reported by snyk. #2989

Merged
merged 1 commit into from
Jul 17, 2023

Conversation

sebr72
Copy link
Contributor

@sebr72 sebr72 commented Jul 13, 2023

Move from openjdk to temurin jdk

@sebr72 sebr72 requested a review from gberaudo July 13, 2023 16:01
@sebr72 sebr72 marked this pull request as ready for review July 13, 2023 16:01
@sebr72 sebr72 enabled auto-merge (rebase) July 13, 2023 16:01
@sebr72 sebr72 requested a review from gberaudo July 13, 2023 18:42
@sebr72 sebr72 self-assigned this Jul 14, 2023
@sebr72 sebr72 added the backport master Backport the pull request to the 'master' branch label Jul 14, 2023
@sebr72 sebr72 requested a review from danduk82 July 17, 2023 06:23
@sebr72 sebr72 merged commit 72e496b into 3.30 Jul 17, 2023
@sebr72 sebr72 deleted the tomcat-without-critical branch July 17, 2023 06:24
@c2c-bot-gis-ci c2c-bot-gis-ci removed the backport master Backport the pull request to the 'master' branch label Jul 17, 2023
@sbrunner
Copy link
Member

Can I have more details about this changes?

@sbrunner
Copy link
Member

sbrunner commented Jul 17, 2023

Which critical error reported by Snyk?

@sbrunner
Copy link
Member

Changing JDK is not be triggered by CVE it wider change, and I didn't see any issue with Snyk:
https://github.com/mapfish/mapfish-print/actions/runs/5571364567/jobs/10176325767

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.

5 participants