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

chore(deps): upgrade gravitee-bom to 7.0.10 #6804

Merged
merged 6 commits into from Mar 2, 2024
Merged

Conversation

jgiovaresco
Copy link
Contributor

@jgiovaresco jgiovaresco commented Mar 1, 2024

Description

To fix vulnerabilities:

⚠️ Jackson 2.16 has removed several deprecated methods. We were massively using DeserializationContext#mappingException, but this one has been deprecated since Jackson 2.9
I used JsonMappingException#from as an alternative to creating the same exception created with the deprecated method.

Additional context

With the upgrade of the gravitee-bom, some Mongo Repositories tests were failing. After a bit of investigation, all failing tests were missing indexes.
I don't really understand what dependencies make this required...


📚  View the storybook of this branch here

Copy link

cypress bot commented Mar 1, 2024

Passing run #786 ↗︎

0 80 8 0 Flakiness 0

Details:

fix(mongo): add missing indexes
Project: APIM UI Tests Commit: 9576f51a8d
Status: Passed Duration: 06:27 💡
Started: Mar 1, 2024 2:42 PM Ended: Mar 1, 2024 2:48 PM

Review all test suite changes for PR #6804 ↗︎

Jackson 2.16 has removed several deprecated methods. We were massively
using DeserializationContext#mappingException, but this one has been
deprecated since Jackson 2.9
As an alternative, I used JsonMappingException#from to create the same
exception that were created with the deprecated method.
With the upgrade of the gravitee-bom, some Mongo Repositories tests were
failing. After a bit of investigation, all failing tests were missing
indexes.
I don't really understand what dependencies make this required...
@jgiovaresco jgiovaresco marked this pull request as ready for review March 1, 2024 15:46
@jgiovaresco jgiovaresco requested a review from a team as a code owner March 1, 2024 15:46
@jgiovaresco jgiovaresco merged commit 48a3610 into master Mar 2, 2024
50 of 51 checks passed
@jgiovaresco jgiovaresco deleted the bump-bom-run-e2e branch March 2, 2024 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants