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

Jackson has problems with similar method signatures #61

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

Dawuid
Copy link

@Dawuid Dawuid commented Mar 3, 2016

Chat-mutiroom example is not working as expected. Jackson raises exception in decoder when try to decode ChatProtocol. Both setUsers methods are too similar.

@elakito
Copy link
Contributor

elakito commented Apr 8, 2016

@Dawuid as the current sample itself doesn't show this problem (when run with mvn jetty:run, which picks jackson 2.4.1), I am wondering with which jackson version you are seeing the problem.

@Dawuid
Copy link
Author

Dawuid commented Apr 13, 2016

I was using the version of March 3 (with atmosphere 2.4.2, jackson-core 2.4.1, jackson-databind 2.4.1, jackson-annotation 2.4.0) deployed in a Wildfly 10. I will try again with last version and come back with the results.

@Dawuid
Copy link
Author

Dawuid commented Apr 21, 2016

Mystery solved. In Wildfly 10, example doesn't work because Wildfly is using internally jackson 2.5.4 (and it's seem that they are publicly exporting such classes in applications classloader, I suppose due to JavaEE json specification). Using directly jackson 2.5+ (tested with 2.5.5, 2.6.6 and 2.7.3) with Jetty, sample doesn't work. So I think Jackson 2.5+ has changed behaviour in bean discovering.

Proposed change works with jetty and jackson 2.4+ (and with Wildfly 9/10).

Extra information: Wildfly 8.2 use jackson 2.4.1 =)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants