You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The problem with depending on jersey is that it's a JAX-RS implementation. If you try to use Xero-Java in a JAX-RS environment using a different JAX-RS implementation like for example RESTEasy then you end up with two JAX-RS implementations on the runtime classpath and which one is being picked up depends on classpath order which is brittle and can lead to various issues.
Expected behavior
No unnecessary dependencies are being declared, especially ones which prove problematic when consuming the library in a JAX-RS application.
The text was updated successfully, but these errors were encountered:
Looks like it might be more complicated than that due to use of jakarta.ws.rs.core.UriBuilder in this library which requires a JAX-RS implementation at runtime.
SDK you're using (please complete the following information):
Describe the bug
Dependencies on jersey were added in ead5159 so that jersey client could be used. Then in e72b4e6#diff-ab9abef9dc24b1ba6e889f45a021ce88ed82095e4d9f95aab63b2cc3f9522bc4 the implementation was switched to using Google HTTP Client Library for Java but the dependency on jersey wasn't removed. This is confirmed by searching for
org.glassfish.jersey
in the repository which only finds that string in the pom and not in any of the java code.The problem with depending on jersey is that it's a JAX-RS implementation. If you try to use Xero-Java in a JAX-RS environment using a different JAX-RS implementation like for example RESTEasy then you end up with two JAX-RS implementations on the runtime classpath and which one is being picked up depends on classpath order which is brittle and can lead to various issues.
Expected behavior
No unnecessary dependencies are being declared, especially ones which prove problematic when consuming the library in a JAX-RS application.
The text was updated successfully, but these errors were encountered: