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.

Sign up for GitHub

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

Jump to bottom

Transition from javax to jakarta #17765

Merged
merged 23 commits into from Jan 16, 2024
Merged

Transition from javax to jakarta #17765

merged 23 commits into from Jan 16, 2024

Conversation

thll
Copy link
Contributor

@thll thll commented Dec 21, 2023 •
edited

Transition away from the javax namespace towards the Jakarta namespace so that we can finally update some of our core dependencies.

  • Replaces most occurrences of javax with jakarta
  • Updates the following dependencies (amongst others):
    • jersey2.37 -> 3.1.5
    • guice 6.0.0 -> 7.0.0
  • Bans a couple of javax dependencies, where possible
  • Adds forbidden API rules to prevent accidental usage of certain javax.* namespaces where banning the respective dependencies is not easily possible.

/prd https://github.com/Graylog2/graylog-plugin-enterprise/pull/6347

@thll
Bump jersey version 2.37 -> 3.1.5
6c7089e
@thll
javax.ws.rs:javax.ws.rs-api -> jakarta.ws.rs:jakarta.ws.rs-api
c39edbe
@thll
Ban javax.ws.rs:javax.ws.rs-api dependency
1f5ba5c
@thll
com.fasterxml.jackson.jaxrs -> com.fasterxml.jackson.jakarta.rs
b5a8216
@thll
javax.validation -> jakarta.validation
e3079ca
@thll
Remove exclusion of hibernate-validator
eedce06
@thll
jackson-module-jsonSchema -> jackson-module-jsonSchema-jakarta
f887cd4
@thll
Bump hibernate-validator 6.2.5.Final -> 8.0.1.Final
ad3d1d6
@thll
javax.annotation-api -> jakarta.annotation-api
5b06346
@thll
Add TODO for banning javax.validation:* dependencies
0cecbb8
@thll
Use jakarta.annotation.* where possible
03fc2e5
@thll
javax.inject -> jakarta.inject
99029f5
@thll
Temporarily disable merging master into PR branch when building
dadbeec
@thll
Add javax.persistence to forbidden APIs
e1ac2be
@thll
Revert "Temporarily disable merging master into PR branch when building"
6ecbf83 
This reverts commit dadbeec.
@thll
Merge remote-tracking branch 'origin/master' into jakarta
adfd3cc
@thll
Fix imports after merge
6d34d40
@thll thll changed the title Hello Jakarta Transition from javax to jakarta Dec 22, 2023
@thll thll requested a review from bernd December 22, 2023 18:32
@thll thll marked this pull request as ready for review December 22, 2023 18:33
@thll thll merged commit 8ce9b3f into master Jan 16, 2024
5 checks passed
@thll thll deleted the jakarta branch January 16, 2024 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bernd bernd bernd approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@thll @bernd