Feature/get delete body

[Ruwann]

Fixes #1689

Changes proposed in this pull request:

• Pass through request body in case of GET and DELETE requests

[coveralls]

coverage: 94.428% (+0.006%) from 94.422% when pulling aea2759 on Ruwann:feature/get-delete-body into 655ea43 on spec-first:main.

[RobbeSneyders]

Thanks @Ruwann!

Should we add a specific test for this behavior? Now it's only implicit in existing tests and we don't test for the GET and similar methods.

Also, I guess this can be a breaking change for users that handled the passed in empty body, so we might want to add this to the smaller breaking changes in the changelog.

[RobbeSneyders]

Can you have a look at my comments above @Ruwann? 🙂

[Ruwann]

Thanks for the comments, I think I got them all :)

[RobbeSneyders]

Thanks Ruwan!

I had another look at the HTTP spec, and seems like there is one method which MUST NOT include a request body, which is the TRACE method. Maybe we should still exclude it.

[Ruwann]

Thanks, I indeed didn't spot that one - nice catch. Had another look and I also think that's the only one, I'll update the PR accordingly!