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

LASB-2748: Resolve critical vulnerabilities reported by Snyk #256

Merged
merged 27 commits into from
Dec 8, 2023

Conversation

mtac50
Copy link
Contributor

@mtac50 mtac50 commented Dec 6, 2023

What

Link to story

  • Updated dependabot configuration.
  • Updated micrometer-registry-cloudwatch version
  • Updated aws-xray-recorder-sdk-spring version.
  • Updated org.apache.cxf:cxf-rt-features-logging version
  • Updated spring-boot
  • Updated gradle
  • Updated the build and deploy pipelines so that the docker image tag using the commit hash.

Checklist

Before you ask people to review this PR:

  • Tests should be passing: ./gradlew test
  • Github should not be reporting conflicts; you should have recently run git rebase master.
  • Avoid mixing whitespace changes with code changes in the same commit. These make diffs harder to read and conflicts more likely.
  • You should have looked at the diff against master and ensured that nothing unexpected is included in your changes.
  • You should have checked that the commit messages say why the change was made.

@mtac50 mtac50 requested review from a team as code owners December 6, 2023 16:41
@mtac50 mtac50 force-pushed the LASB-2748-Fix-snyk-dependencies branch from cff819b to 8c83341 Compare December 6, 2023 23:47
lkm0287
lkm0287 previously approved these changes Dec 7, 2023
… and aws-java-sdk-s3 versions to remove vulnerability.
Copy link

@Ross-Nation Ross-Nation left a comment

Choose a reason for hiding this comment

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

I think everything looks good to my eyes, I couldn't see any glaringly obvious issues and I think I'm getting a better understanding of how the pipelines hook up :)

@mtac50 mtac50 merged commit 0127cd9 into master Dec 8, 2023
2 checks passed
@mtac50 mtac50 deleted the LASB-2748-Fix-snyk-dependencies branch December 8, 2023 15:31
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