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

Updated dependencies and resolving CVEs, bumped SpringBoot to 3.3.5 #6449

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

madduci
Copy link
Contributor

@madduci madduci commented Nov 6, 2024

Updated:

  • SpringBoot
  • Spring Components
  • Jackson
  • Log4j
  • Logback
  • Swagger UI
  • Bootstrap
  • OpenTelemetry Instrumentation
  • Hibernate
  • Testcontainers
  • MySQL Connector
  • PostgreSQL driver
  • Oracle11 driver
  • ElasticSearch
  • GraphQL

@jamesagnew
Copy link
Collaborator

Hey @madduci , thanks for the contribution!

Two quick initial comments:

@madduci
Copy link
Contributor Author

madduci commented Nov 6, 2024 via email

@jamesagnew
Copy link
Collaborator

@madduci it looks like there is still one still failing CI test: GraphQLR4Test. It's failing with a super weird error, I'm having a look to see if I can understand what's up with that.

@madduci
Copy link
Contributor Author

madduci commented Nov 7, 2024

@jamesagnew thank you very much, I don't have any clue what could be the problem in the test

@jamesagnew
Copy link
Collaborator

@madduci FYI I fixed that graphql issue and pushed it in.

It turns out that we have a team internally over here who are working on bumping hibernate already (but not boot or - and they've already rules out the timestampdiff approach for resolving that one issue because it's not cross-database compatible.

I'm going to propose that we let that work get merged and sync it into your branch just to avoid any extra work - We'll get this merged, but it'll be in a few days probably.

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