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

Adds failing test for Github issue 518. #519

Merged
merged 3 commits into from
Nov 16, 2021
Merged

Conversation

schmist
Copy link
Contributor

@schmist schmist commented Nov 13, 2021

See #518 for a description.

@dinomite
Copy link
Member

Would you fill out the CLA and email a scan/photo of the result to info at fasterxml dot com?

https://github.com/FasterXML/jackson/blob/master/contributor-agreement.pdf

@dinomite dinomite added the cla-needed PR looks good (although may also require code review), but CLA needed from submitter label Nov 15, 2021
@dinomite dinomite self-assigned this Nov 15, 2021
@schmist
Copy link
Contributor Author

schmist commented Nov 16, 2021

Would you fill out the CLA and email a scan/photo of the result to info at fasterxml dot com?

https://github.com/FasterXML/jackson/blob/master/contributor-agreement.pdf

Done.

@dinomite
Copy link
Member

Great! Now, as for fixing the bug…I recall having looked and there was some reason why it wasn't easy, but given I don't remember why, I bet a lot has changed both in this module & databind, so might be solvable now.

Thanks for supplying the test!

@dinomite dinomite merged commit a528c41 into FasterXML:2.13 Nov 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-needed PR looks good (although may also require code review), but CLA needed from submitter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants