DELETE request drops the body

[AlexGhiondea]

It looks like for a DELETE request the body is not parsed.

I looked at the history but couldn't figure out if there was a reason why for a DELETE the body is not parsed/matched?

I am writing tests that depend on the DELETE request sending a body and this makes it tricky to test.

The fix can be as easy as flipping this to true, however I am not sure what the side effects could be.
https://github.com/WireMock-Net/WireMock.Net/blob/master/src/WireMock.Net/Util/BodyParser.cs#L36

[StefH]

I see your point.
For mocking and testing purposes, it would be better to just allow all, or make this configurable with settings.

[StefH]

I've added a AllowBodyForAllHttpMethods property in the settings.

Please test MyGet version:
WireMock.Net.1.0.32-ci-11945.nupkg

[StefH]

@AlexGhiondea ; did you have time to test?

[StefH]

@AlexGhiondea ; did you have time to test this preview version?

[AlexGhiondea]

Hey
I apologize for the delay.I just saw your messages.

Thanks for adding that!

I will give it a try later today!!

Thanks,
Alex

[StefH]

@AlexGhiondea Did you have time to test this?

[StefH]

@AlexGhiondea Can you please test and report your findings?

[AlexGhiondea]

@StefH I didn’t get around to test this yet. I will give it a try at lunch today!

[AlexGhiondea]

@StefH I wasn't able to get the package from the feed, so I cloned and built locally your PR. With that, the tests that require a body for a DELETE operation work if you specify that setting!

Thanks for making the change! I will update my version once you release!!

[StefH]

Thank you for testing this. I will merge this PR to master, keep a watch on the official NuGet packages.