Skip to content

Sanitize \r in request body.#388

Merged
umeshp7 merged 4 commits intodevelopfrom
issue-350
Oct 28, 2020
Merged

Sanitize \r in request body.#388
umeshp7 merged 4 commits intodevelopfrom
issue-350

Conversation

@webholik
Copy link
Contributor

Fixes #350.

@webholik webholik requested a review from umeshp7 October 21, 2020 18:26
@umeshp7
Copy link
Contributor

umeshp7 commented Oct 25, 2020

Can we add unit tests for \n, \r etc. wherever applicable?

@umeshp7 umeshp7 merged commit 8412cad into develop Oct 28, 2020
@umeshp7 umeshp7 deleted the issue-350 branch December 15, 2020 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

js-fetch and nodejs-axios fail dealing line breaks(\r\n) in body

2 participants