-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Application requires explicit inclusion of fasterxml #1
Comments
Thank you so much for the issue! I'll get a fix out right away. And thanks for the kudos! Suggestions are welcome! |
As far as the dependency - I'm a bit confused - all the test cases work without it. Might this be something specific to your app? (e.g. the Jackson lib we're using conflicts with one of yours perhaps?) Not sure... |
I've tried really hard to reproduce this in a test app. I even tried including a different jackson, and it didn't fail. We have a bunch of dependencies in our app, so that is likely. Here's one which looks like it may be a problem.
|
I just looked in our build config. This is listed as an explicit compile-time dependency: <dependency>
<groupId>com.fasterxml.jackson.core</groupId>
<artifactId>jackson-databind</artifactId>
<version>${jackson.version}</version>
</dependency> (where I ran
which lists the If you remove the |
Ah, yes - if you have Jackson < 2 in your project already, this will likely cause a conflict. I'll update the docs to reflect this, thank you! |
I just updated the readme to cover this: https://github.com/jwtk/jjwt/blob/master/README.md I believe this to be a sufficient solution to this issue since I can't change the project's dependency settings (since they already explicitly reference a compile-time dependency). Thoughts? |
Here is the dependency tree. It actually was a runtime error rather than a compile or build error. [INFO] org.myapp.www:phoenix:war:1.0-SNAPSHOT |
And while I have your attention, now that I realize who is on the other side of this, thank you for your videos on REST APIs. You've strongly influenced how I build APIs. 👍 :) |
Updated the docs to reflect the runtime nature. And thanks for the kind words - it's always nice to know if I've been helpful to others! |
Closing this issue per updated docs. If you feel a need to re-open, please feel free! |
Merge Updates from Upstream Master
I'm curious to watch the videos on REST APIs that was mentioned in @mattjonesorg last comment. Will someone add a link as a comment? |
@axiopisty -- Here's one of my favorites: https://www.youtube.com/watch?v=hdSrT4yjS1g. And here's the written form: https://stormpath.com/blog/fundamentals-rest-api-design. |
For some reason when using the gradle import I get the following:
When I inspected my dependencies it seems that jsonwebtoken is getting all three files, so I'm confused as to why it's finding duplicates |
@GMSyrimis I solved it with this. |
So, when I included the section mentioned in the wiki into my pom.xml of my appengine application (Ninja Framework):
and ran the application, I got the following stack trace, repeatedly.
The resolution of this was to manually include the dependency for fasterxml.
I'm not sure why this is the case, but wanted to document it here just in case anybody else runs into this in the future.
BTW, This looks like an AWESOME library!
The text was updated successfully, but these errors were encountered: