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

Upgrade to springboot 3.0.1. #473

Closed
wants to merge 1 commit into from

Conversation

lukedegruchy
Copy link
Contributor

No description provided.

…odError: Receiver class org.springframework.boot.autoconfigure.jersey.JerseyAutoConfiguration$JerseyWebApplicationInitializer does not define or inherit an implementation of the resolved method 'abstract void onStartup(javax.servlet.ServletContext)' of interface org.springframework.web.WebApplicationInitializer."
@XcrigX
Copy link
Contributor

XcrigX commented Jan 9, 2023

I'm wondering if this wouldn't be safer to do in the base HAPI project? As in - have the base HAPI project import the SpringBoot BOM, and rely on the curated versioning in it so that everything is tested with the updated Spring versions. It seems dangerous to mix-n-match core spring and spring boot versions.

@jkiddo
Copy link
Collaborator

jkiddo commented Jan 10, 2023

@jkiddo
Copy link
Collaborator

jkiddo commented Jan 10, 2023

First requirement is to have the spring core dependency in hapi base bumped to 6

@jkiddo
Copy link
Collaborator

jkiddo commented Dec 4, 2023

@lukedegruchy looks like #618 superseeds this PR - correct?

@lukedegruchy
Copy link
Contributor Author

lukedegruchy commented Dec 4, 2023

@lukedegruchy looks like #618 superseeds this PR - correct?

@jkiddo This MR is really old and I'll close it momentarily.

If I were to speculate, I would say James' intention in his PR is to pull in hapi-fhir's springboot version, but I can't say 100%.

Please perform your due diligence when testing the other PR.

@lukedegruchy
Copy link
Contributor Author

Closing since this PR is obsolete.

@patrick-werner patrick-werner deleted the ld-20230109-spring-boot-3-0-1 branch April 23, 2024 12:33
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.

None yet

3 participants