Allow 64 bit numbers in JSON #251

Closed
opened 2025-12-29 08:24:46 +01:00 by adam · 6 comments
Owner

Originally created by @ezarko on GitHub (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.

Originally created by @ezarko on GitHub (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.
adam added the bug label 2025-12-29 08:24:46 +01:00
adam closed this issue 2025-12-29 08:24:46 +01:00
Author
Owner

@StefH commented on GitHub (Feb 20, 2020):

Thank you for reporting. I will take a look.

@StefH commented on GitHub (Feb 20, 2020): Thank you for reporting. I will take a look.
Author
Owner

@StefH commented on GitHub (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 commented on GitHub (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)
Author
Owner

@ezarko commented on GitHub (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.

@ezarko commented on GitHub (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.
Author
Owner

@StefH commented on GitHub (Feb 25, 2020):

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

@StefH commented on GitHub (Feb 25, 2020): I will just merge this fix into master and make a new Docker. Keep a watch on dockerhub
Author
Owner

@ezarko commented on GitHub (Mar 12, 2020):

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

@ezarko commented on GitHub (Mar 12, 2020): I have verified this in 1.1.10. Works great. Thanks!
Author
Owner

@StefH commented on GitHub (Mar 13, 2020):

Thanks for checking.

@StefH commented on GitHub (Mar 13, 2020): Thanks for checking.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/WireMock.Net#251