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

fix: java dependency vulnerabilities #4460

Merged
merged 2 commits into from Mar 22, 2022
Merged

fix: java dependency vulnerabilities #4460

merged 2 commits into from Mar 22, 2022

Conversation

ghost
Copy link

@ghost ghost commented Mar 21, 2022

Checklist

  • The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
  • Links to related issues (if applicable)
  • Tests for the changes have been added/updated (if applicable)
  • Docs related to the changes have been added/updated (if applicable)

@ghost ghost marked this pull request as draft March 21, 2022 18:09
@ghost ghost marked this pull request as ready for review March 21, 2022 18:09
@github-actions
Copy link

github-actions bot commented Mar 21, 2022

Unit Test Results (build & test)

  92 files  ±0    92 suites  ±0   26m 27s ⏱️ + 8m 24s
674 tests ±0  615 ✔️ ±0  59 💤 ±0  0 ±0 

Results for commit 55e163c. ± Comparison against base commit c5f1d2c.

♻️ This comment has been updated with latest results.

@ghost ghost marked this pull request as draft March 21, 2022 22:17
@ghost ghost marked this pull request as ready for review March 22, 2022 21:15
Copy link
Contributor

@dexter-mh-lee dexter-mh-lee left a comment

Choose a reason for hiding this comment

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

LGTM!

@dexter-mh-lee dexter-mh-lee merged commit 4476356 into datahub-project:master Mar 22, 2022
@ghost ghost deleted the ecc--PLT-2-fix-java-dep-vulns branch March 22, 2022 21:28
maggiehays pushed a commit to maggiehays/datahub that referenced this pull request Aug 1, 2022
* fix: java dependency vulnerabilities

* Upgrade org.hibernate:hibernate-validator (PLT-2)
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.

1 participant