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

Bump finatra to 20.5.0 and jackson 2.10 #48

Closed
wants to merge 2 commits into from

Conversation

kimxogus
Copy link

Jackson 2.11 blocker swagger-api/swagger-core#3554

@kimxogus kimxogus closed this May 29, 2020
@jakehschwartz
Copy link
Owner

@kimxogus Why did you close? Was it not working as expected?

@mgd43b
Copy link

mgd43b commented Jun 3, 2020

@jakehschwartz - Finatra is now using Jackson 2.11 - which I think means this might be more involved, as the io.swagger libs are still on 2.10.1, and they might need upgraded.

@kimxogus
Copy link
Author

kimxogus commented Jun 3, 2020

no it wasn't.
with jackson 2.10, it causes exception

@jakehschwartz
Copy link
Owner

Got it, I missed the issue referenced in the original post. Sounds like they are hoping to update it soon, so that's good. In the meantime, I can try to make sure we can move to the new swagger-core repository without issues and then update all the jackson stuff at once

@jakehschwartz
Copy link
Owner

jakehschwartz commented Jun 3, 2020

Right which it doesnt because they were on Jackson 2.9 in previous version of Finatra. Fun 🤣

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.

3 participants