yakbak icon indicating copy to clipboard operation
yakbak copied to clipboard

fix(comment-escaping): Escape comment delimiters in `req.path`

Open jordanrastrick opened this issue 6 years ago • 1 comments

(and occurring multiply)

Currently a req.path containing */ (* is a valid URL character) will on recording terminate the headers' comment, leaving a broken tape. Similarly if a /* or */ occurs more than once the additional sequences are not escaped.

jordanrastrick avatar May 09 '18 07:05 jordanrastrick

Thank you for submitting this pull request, however I do not see a valid CLA on file for you. Before we can merge this request please visit https://yahoocla.herokuapp.com/ and agree to the terms. Thanks! :smile:

yahoocla avatar May 09 '18 07:05 yahoocla