ParamMatcher failed when using parameter with comma #467

New Issue

Closed

opened 2025-12-29 15:24:30 +01:00 by adam · 16 comments

adam commented 2025-12-29 15:24:30 +01:00

Owner

Copy Link

Originally created by @geovanegodoi on GitHub (Nov 24, 2022).

Originally assigned to: @StefH on GitHub.

Describe the bug

I'm working on a system that consumes a 3rd party API which accepts as querystring a SQL statement.
For example, to query some data from any table just build the URI like this:

• http://host/api/queryservice?query=SELECT id FROM table WHERE id = 1
  While trying to mock this API, I realized that whenever I tried to mock a statement using comma, the ParamMatcher fails.
  For example, SELECT id, value FROM table WHERE id = 1.
  And I received a 404 (NotFound) error code.

Expected behavior:

I expected the ParamMatcher has succeded, and returned 200 (OK).

Test to reproduce

1 - Create an API which receives as querystring a SQL statement
2 - Mock that API using "WithParam" function
3 - Test creating a statement which has a 'comma' within, "SELECT id, value FROM table WHERE id = 1"

Other related info

Provide additional information if any.

Originally created by @geovanegodoi on GitHub (Nov 24, 2022). Originally assigned to: @StefH on GitHub. ### Describe the bug I'm working on a system that consumes a 3rd party API which accepts as querystring a SQL statement. For example, to query some data from any table just build the URI like this: - `http://host/api/queryservice?query=SELECT id FROM table WHERE id = 1` While trying to mock this API, I realized that whenever I tried to mock a statement using comma, the ParamMatcher fails. For example, `SELECT id, value FROM table WHERE id = 1`. And I received a 404 (NotFound) error code. ### Expected behavior: I expected the ParamMatcher has succeded, and returned 200 (OK). ### Test to reproduce 1 - Create an API which receives as querystring a SQL statement 2 - Mock that API using "WithParam" function 3 - Test creating a statement which has a 'comma' within, "SELECT id, value FROM table WHERE id = 1" ### Other related info Provide additional information if any.

adam added the question label 2025-12-29 15:24:30 +01:00

adam closed this issue 2025-12-29 15:24:31 +01:00

adam commented 2025-12-29 15:24:31 +01:00

Author

Owner

Copy Link

@StefH commented on GitHub (Nov 24, 2022):

What you can try:

1️⃣
url-escape the comma

2️⃣
Set https://github.com/WireMock-Net/WireMock.Net/blob/master/src/WireMock.Net/Settings/WireMockServerSettings.cs#L267 to QueryParameterMultipleValueSupport.NoComma

@StefH commented on GitHub (Nov 24, 2022): What you can try: :one: url-escape the comma :two: Set https://github.com/WireMock-Net/WireMock.Net/blob/master/src/WireMock.Net/Settings/WireMockServerSettings.cs#L267 to `QueryParameterMultipleValueSupport.NoComma`

adam commented 2025-12-29 15:24:31 +01:00

Author

Owner

Copy Link

@geovanegodoi commented on GitHub (Nov 24, 2022):

Thanks for reply,

I've downloaded the latest nuget version (1.5.10) but there isn't this property.

Am I doing something wrong?

@geovanegodoi commented on GitHub (Nov 24, 2022): Thanks for reply, I've downloaded the latest nuget version (1.5.10) but there isn't this property.  Am I doing something wrong?

adam commented 2025-12-29 15:24:32 +01:00

Author

Owner

Copy Link

@StefH commented on GitHub (Nov 24, 2022):

No. Nothing is wrong on your end

I just realized that I need to release a new version which includes this.

I'll do that now.

@StefH commented on GitHub (Nov 24, 2022): No. Nothing is wrong on your end I just realized that I need to release a new version which includes this. I'll do that now.

adam commented 2025-12-29 15:24:33 +01:00

Author

Owner

Copy Link

@geovanegodoi commented on GitHub (Nov 24, 2022):

Thanks, @StefH

I've just downloaded the new release, and now the property is ok.

When you said "url-escape the comma", could you show an example?

I tried somethings, but I still got the error.

@geovanegodoi commented on GitHub (Nov 24, 2022): Thanks, @StefH I've just downloaded the new release, and now the property is ok. When you said "url-escape the comma", could you show an example? I tried somethings, but I still got the error.

adam commented 2025-12-29 15:24:33 +01:00

Author

Owner

Copy Link

@StefH commented on GitHub (Nov 25, 2022):

It could be that Url escaping does indeed nor work.

Then you need to use option 2.

Can this issue be closed?

@StefH commented on GitHub (Nov 25, 2022): It could be that Url escaping does indeed nor work. Then you need to use option 2. Can this issue be closed?

adam commented 2025-12-29 15:24:34 +01:00

Author

Owner

Copy Link

@geovanegodoi commented on GitHub (Nov 25, 2022):

I'm still facing the issue.

The API I'm trying to mock is this one:
https://docs.oracle.com/en/cloud/saas/service/18a/cxsvc/op-services-rest-connect-v1.4-queryresults-get.html

It's a public Oracle REST API, to query entities from database.
When I try to mock this request I get a 404 error.

@geovanegodoi commented on GitHub (Nov 25, 2022): I'm still facing the issue. The API I'm trying to mock is this one: https://docs.oracle.com/en/cloud/saas/service/18a/cxsvc/op-services-rest-connect-v1.4-queryresults-get.html It's a public Oracle REST API, to query entities from database. When I try to mock this request I get a 404 error. 

adam commented 2025-12-29 15:24:34 +01:00

Author

Owner

Copy Link

@StefH commented on GitHub (Nov 30, 2022):

https://github.com/WireMock-Net/WireMock.Net/pull/854

@StefH commented on GitHub (Nov 30, 2022): https://github.com/WireMock-Net/WireMock.Net/pull/854

adam commented 2025-12-29 15:24:34 +01:00

Author

Owner

Copy Link

@StefH commented on GitHub (Nov 30, 2022):

@gigiogodoi
Can you please provide a full working c# code example which simulates your issue? Because I cannot reproduce it using unit tests.

@StefH commented on GitHub (Nov 30, 2022): @gigiogodoi Can you please provide a full working c# code example which simulates your issue? Because I cannot reproduce it using unit tests.

adam commented 2025-12-29 15:24:34 +01:00

Author

Owner

Copy Link

@geovanegodoi commented on GitHub (Nov 30, 2022):

@StefH
Follow an example repository to reproduce the error, using XUnit and .NET6 .

https://github.com/gigiogodoi/UnitTestWithWiremock

Inside it, there are 2 test cases, one failing because it is using a query with comma.

@geovanegodoi commented on GitHub (Nov 30, 2022): @StefH Follow an example repository to reproduce the error, using XUnit and .NET6 . https://github.com/gigiogodoi/UnitTestWithWiremock Inside it, there are 2 test cases, one failing because it is using a query with comma. 

adam commented 2025-12-29 15:24:34 +01:00

Author

Owner

Copy Link

@StefH commented on GitHub (Nov 30, 2022):

Thanks a lot for this unit test. I did find the issue and it's fixed in preview version 1.5.11-ci-16748.

@StefH commented on GitHub (Nov 30, 2022): Thanks a lot for this unit test. I did find the issue and it's fixed in preview version `1.5.11-ci-16748`.

adam commented 2025-12-29 15:24:35 +01:00

Author

Owner

Copy Link

@geovanegodoi commented on GitHub (Nov 30, 2022):

@StefH
Could you add a testcase to validate a querystring like this ?

query=select id, name, value from table where id in (1, 2, 3, 4, 5)

@geovanegodoi commented on GitHub (Nov 30, 2022): @StefH Could you add a testcase to validate a querystring like this ? **query=select id, name, value from table where id in (1, 2, 3, 4, 5)**

adam commented 2025-12-29 15:24:35 +01:00

Author

Owner

Copy Link

@StefH commented on GitHub (Nov 30, 2022):

Yes.

Works fine.

https://github.com/WireMock-Net/WireMock.Net/blob/stef-849/test/WireMock.Net.Tests/WireMockServerTests.WithParam.cs

@StefH commented on GitHub (Nov 30, 2022): Yes. Works fine. https://github.com/WireMock-Net/WireMock.Net/blob/stef-849/test/WireMock.Net.Tests/WireMockServerTests.WithParam.cs

adam commented 2025-12-29 15:24:35 +01:00

Author

Owner

Copy Link

@StefH commented on GitHub (Dec 2, 2022):

@gigiogodoi Is it ok?

@StefH commented on GitHub (Dec 2, 2022): @gigiogodoi Is it ok?

adam commented 2025-12-29 15:24:36 +01:00

Author

Owner

Copy Link

@StefH commented on GitHub (Dec 3, 2022):

I'll merge the PR.

@StefH commented on GitHub (Dec 3, 2022): I'll merge the PR.

adam commented 2025-12-29 15:24:36 +01:00

Author

Owner

Copy Link

@geovanegodoi commented on GitHub (Dec 3, 2022):

I was not able to download the preview version, because it was not releated in the nuget package manager.

But I will try once you release a new version.

@geovanegodoi commented on GitHub (Dec 3, 2022): I was not able to download the preview version, because it was not releated in the nuget package manager. But I will try once you release a new version.

adam commented 2025-12-29 15:24:37 +01:00

Author

Owner

Copy Link

@StefH commented on GitHub (Dec 3, 2022):

previews are here: https://github.com/WireMock-Net/WireMock.Net/wiki/MyGet-preview-versions

But I'll release an official version today on nuget.

@StefH commented on GitHub (Dec 3, 2022): previews are here: https://github.com/WireMock-Net/WireMock.Net/wiki/MyGet-preview-versions But I'll release an official version today on nuget.

Sign in to join this conversation.

No Branch/Tag Specified

Branches Tags

master

mime-part-matcher

version-2.x

MappingSerializer

1341-mapping-json

stef-aspire-tests-updates

bug/973-TinyMapper

bug/1149-AppendGuidToSavedMappingFile

feature/1150-DockerImageVersion

61

stef-1108

stef-1097

stef-1083-MessageOptions_Type_Conflict

stef-1062-logger

stef-IgnoreOpenApiErrors

stef-928-TypeLoadException-FluentAssertions-net472

nunit

stef-849

stef-847-regex-questionmark

http_verb

WireMockServerContext

webapp

ai

CommandLineArgumentsParser

1.23.0

1.22.0

1.21.0

1.20.0

1.19.0

1.18.0

1.17.0

1.16.0

1.15.0

1.14.0

1.13.0

1.12.0

1.11.2

1.11.0

1.10.1

1.10.0

1.9.1

1.9.0

1.8.18

1.8.17

1.8.16

1.8.15

1.8.14

1.8.13

1.8.12

1.8.11

1.8.10

1.8.9

1.8.8

1.8.7

1.8.6

1.8.5

1.8.3

1.8.2

1.8.1

1.8.0

1.7.4

1.7.3

1.7.2

1.7.1

1.7.0

1.6.12

1.6.11

1.6.10

1.6.9

1.6.8

1.6.7

1.6.6

1.6.5

1.6.4

1.6.3

1.6.2

1.6.1

1.6.0

1.5.62

1.5.61

1.5.60

1.5.59

1.5.58

1.5.57

1.5.56

1.5.55

1.5.54

1.5.53

1.5.52

1.5.51

1.5.50

1.5.49

1.5.48

1.5.47

1.5.46

1.5.45

1.5.44

1.5.43

1.5.42

1.5.41

1.5.40

1.5.39

1.5.38

1.5.37

1.5.36

1.5.35

1.5.34

1.5.33

1.5.32

1.5.31

1.5.30

1.5.29

1.5.28

1.5.27

1.5.26

1.5.25

1.5.24

1.5.23

1.5.22

1.5.21

1.5.20

1.5.19

1.5.18

1.5.17

1.5.16

1.5.15

1.5.14

1.5.13

1.5.12

1.5.11

1.5.10

1.5.9

1.5.8

1.5.7

1.5.6

1.5.5

1.5.4

1.5.3

1.5.2

1.5.1

1.5.0

1.4.43

1.4.42

1.4.41

1.4.40

1.4.39

1.4.38

1.4.37

1.4.36

1.4.35

1.4.34

1.4.33

1.4.32

1.4.31

1.4.30

1.4.29

1.4.28

1.4.27

1.4.26

1.4.25

1.4.24

1.4.23

1.4.22

1.4.21

1.4.20

1.4.19

1.4.18

1.4.17

1.4.16

1.4.15

1.4.14

1.4.13

1.4.12

1.4.11

1.4.10

1.4.9

1.4.8

1.4.7

1.4.6

1.4.5

1.4.4

1.4.3

1.4.2

1.4.1

1.4.0

1.3.10

1.3.9

1.3.8

1.3.6

1.3.5

1.3.4

1.3.3

1.3.2

1.3.1

1.3.0

1.2.18

1.2.17

1.2.16

1.2.15

1.2.14

1.2.13

1.2.12

1.2.11.0

1.2.10

1.2.9.0

1.2.8.0

1.2.7.0

1.2.6.0

1.2.5.0

1.2.4.0

1.2.3.0

1.2.2.0

1.2.1.0

1.2.0.0

1.1.10

1.1.9.0

1.1.8.0

1.1.7.0

1.1.6.0

1.1.5.0

1.1.3.0

1.1.2.0

1.1.1.0

1.1.0.0

1.0.43.0

1.0.42.0

1.0.41.0

1.0.40.0

1.0.39.0

1.0.38.0

1.0.37.0

1.0.36.0

1.0.35.0

1.0.34.0

1.0.33.0

1.0.32.0

1.0.31.0

1.0.29.0

1.0.28.0

1.0.25.0

1.0.24.0

1.0.23.0

1.0.22.0

1.0.21.0

1.0.20.0

1.0.19.0

1.0.18.0

1.0.17.0

1.0.16.0

1.0.15.0

1.0.14.0

1.0.13.0

1.0.12.0

1.0.11.0

1.0.10.0

1.0.9.0

1.0.8.0

1.0.7.0

1.0.6.1

1.0.6

1.0.5

1.0.4.21

1.0.4.20

1.0.4.19

1.0.4.18

1.0.4.17

1.0.4.16

1.0.4.15

1.0.4.14

1.0.4.13

1.0.4.12

1.0.4.11

1.0.4.10

1.0.4.9

1.0.4.8

1.0.4.7

1.0.4.6

1.0.4.5

1.0.4.4

1.0.4.3

1.0.4.2

1.0.4.1

1.0.4.0

1.0.3.20

1.0.3.19

1.0.3.18

1.0.3.17

1.0.3.16

1.0.3.15

1.0.3.14

1.0.3.13

1.0.3.12

1.0.3.11

1.0.3.10

1.0.3.9

1.0.3.8

1.0.3.7

1.0.3.6

1.0.3.5

1.0.3.4

1.0.3.3

1.0.3.2

1.0.3.1

1.0.3.0

1.0.2.13

1.0.2.12

1.0.2.11

1.0.2.10

1.0.2.9

1.0.2.8

1.0.2.7

1.0.2.6

1.0.2.5

1.0.2.4

1.0.2.1

1.0.2.0

1.0.1.5

1.0.1.3

1.0.1.2

1.0.1.1

1.0.0.0

Labels

Clear labels

bug

bug

doc

doc

duplicate

feature

feature

invalid

pull-request

Mirrored from GitHub Pull Request

question

wontfix

No Label question

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

adam

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/WireMock.Net-wiremock#467