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 exception message on failure to create JakartaXmlBindAnnotationIntrospector #5811

Merged

Conversation

Spikhalskiy
Copy link
Contributor

@Spikhalskiy Spikhalskiy commented Nov 28, 2024

Commit 62ba351 addressed the wrong class being used in Jersey 3.0.x but didn't adjust an error message accordingly. This PR fixes this.

@senivam
Copy link
Contributor

senivam commented Nov 29, 2024

just FYI, the line you are changing (along with the whole Jackson in the **/jackson/jaxrs/** package) is repackaged from here

@Spikhalskiy
Copy link
Contributor Author

Spikhalskiy commented Nov 29, 2024

@senivam I’m not sure what the implication is. In that code version the class name in the message is aligned with the class used. After 62ba351 it doesn’t and needs to be adjusted.

@senivam
Copy link
Contributor

senivam commented Nov 29, 2024

@Spikhalskiy sure thing. That change was also made to the repackaged code. I just wanted to let you know that its origin is not ours. For us, it's important to keep track of those changes not to be re-written on the next version update. For you, it's just informative :)

@senivam senivam merged commit 6ab1ad1 into eclipse-ee4j:3.0 Dec 2, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants