Parse body of PATCH requests

nbs-system / naxsi

NAXSI is an open-source, high performance, low rules maintenance WAF for NGINX

GNU General Public License v3.0

4.8k stars 606 forks source link

Parse body of PATCH requests #426

Closed calve closed 6 years ago

calve commented 6 years ago

Title say it all; allow to parse body of PATCH requests

should fix https://github.com/nbs-system/naxsi/issues/358

buixor commented 6 years ago

Thanks for the MR, giving it a look :)

buixor commented 6 years ago

@calve would you mind adding some unit tests ? In the meanwhile I'm going to check if there are some pitfalls we might take care of when dealing with PATCH method.

calve commented 6 years ago

Thanks for the merge. Sorry I did not took time to add proper tests. On the other hand, there is no test for PUT requests aswell. Shall we copy/paste the tests for POST or factorise them ?

jvoisin commented 6 years ago

Yup, this is the way to go :)