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

feign-soap change from javax to jakarta #1918

Closed
ambikurup opened this issue Jan 25, 2023 · 9 comments
Closed

feign-soap change from javax to jakarta #1918

ambikurup opened this issue Jan 25, 2023 · 9 comments
Labels
proposal Proposed Specification or API change

Comments

@ambikurup
Copy link

For spring boot 3 support, will feign-soap using javax be changed to jakarta. Currently using 12.1 version I'm getting

javax.xml.bind.JAXBException Implementation of JAXB-API has not been found on module path or classpath

I added jaxb dependency but still getting same error.

@sandalovsergey
Copy link

Have a same problem.. @velo can you please take a look?

I've found this one #1700 but it still use javax packages instead jakarta packages. F.e. see this: https://github.com/OpenFeign/feign/blob/12.1/jaxb/src/main/java/feign/jaxb/JAXBContextFactory.java#L21

It is from jaxb-api (javax.xml.bind:jaxb-api:2.3.1) but afaik shoud be from bind-api (jakarta.xml.bind:jakarta.xml-bind).

Did i make a mistake?

@velo
Copy link
Member

velo commented Feb 7, 2023

Probably a new module is required for Jakarta change

@sandalovsergey
Copy link

@velo thanks for the answer!

Did I understood correctly that I can create a PR with new module(f.e feign-jaxb-jakarta), where instead of javax.xml.bind-packages, there will be jakarta.xml.bind-packages ?

@velo
Copy link
Member

velo commented Feb 8, 2023

If you can do that, that would be amazing.

Someone already did it for jaxrs, so seems reasonable

@ambikurup
Copy link
Author

can you please let me know when will be the new module published and ready to use?

@kdavisk6 kdavisk6 added the proposal Proposed Specification or API change label May 24, 2023
@kdavisk6
Copy link
Member

Feign Jakarta is live on Maven Central since 12.3

@virtual-machinist
Copy link
Contributor

@kdavisk6 Feign Jakarta contains only the class feign.jaxrs.JakartaContract, it does not have anything found e.g. in soap like SOAPEncoder. What I believe to be the original reason for opening this issue hasn't been solved yet.

Since there's already jaxb-jakarta module, I can add soap-jakarta based on soap. I've created a fork with the new module and can add a PR if it's okay.

@EarthCitizen
Copy link

@kdavisk6 @velo

As mentioned above by @virtual-machinist, this original problem of this issue is not solved by the release of module jaxb-jakarta as soap still uses the old APIs. There needs to be a soap-jakarta as laid out by @virtual-machinist .

@kistlers
Copy link
Contributor

kistlers commented Jul 20, 2023

@virtual-machinist @EarthCitizen
I already implemented the feign-soap-jakarta module. And it is already merged and released with feign 12.4!. See #2094

I just included the new module this week in a project at work, where i replaced feign-soap by the new jakarta module and managed to replace old jaxb by jakarta dependencies.

Please go ahead and use it. I'd gladly hear feedback if there are any bugs or missing features.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
proposal Proposed Specification or API change
Projects
None yet
Development

No branches or pull requests

7 participants