naz / swagger-express-validator

Lightweight requrest/response validation middleware based on swagger/OpenAPI schema
MIT License
56 stars 24 forks source link

Shouldn't mutate original data in res.end #51

Open kanavarora opened 5 years ago

kanavarora commented 5 years ago

In res.end we seem to be converting the incoming data to json to validate the request/response. However, the ideally the return from res.end should just be the original data. In all such cases, this MR just returns the original data, and removes the unnecessary conversion in stringify call in senddata. Have added test cases for pure string and boolean responses too. (should add for number as well). Note: boolean and number was already working. Was just failing for string. Issue Described here: https://github.com/gargol/swagger-express-validator/issues/50