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

Allow 64 bit numbers in JSON #425

Closed
ezarko opened this issue Feb 20, 2020 · 6 comments
Closed

Allow 64 bit numbers in JSON #425

ezarko opened this issue Feb 20, 2020 · 6 comments
Labels

Comments

@ezarko
Copy link

ezarko commented Feb 20, 2020

I noticed in testing that if I add a key to a JSON body with an integer value greater than the maxint for a 32 bit integer, then a LinqMarcher on the body will fail, even if it is not checking that field. I would guess that it is no longer parsing the body correctly.

@StefH
Copy link
Collaborator

StefH commented Feb 20, 2020

Thank you for reporting. I will take a look.

@StefH
Copy link
Collaborator

StefH commented Feb 22, 2020

@ezarko Can you please test WireMock.Net.1.1.8-ci-12754.nupkg version from MyGet (https://github.com/WireMock-Net/WireMock.Net/wiki/MyGet-preview-versions)

@StefH StefH added the bug label Feb 25, 2020
@ezarko
Copy link
Author

ezarko commented Feb 25, 2020

Would it be possible for you to provide a test docker image for https://github.com/WireMock-Net/WireMock.Net-docker? Or at least a Dockerfile I could use to build the same? I tried to spin one up myself, but it is taking me longer than I have to devote to this today.

@StefH
Copy link
Collaborator

StefH commented Feb 25, 2020

I will just merge this fix into master and make a new Docker. Keep a watch on dockerhub

@StefH StefH closed this as completed Feb 25, 2020
@ezarko
Copy link
Author

ezarko commented Mar 12, 2020

I have verified this in 1.1.10. Works great. Thanks!

@StefH
Copy link
Collaborator

StefH commented Mar 13, 2020

Thanks for checking.

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

No branches or pull requests

2 participants