jmeter-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [jmeter] jloisel commented on pull request #616: Fix #63255
Date Mon, 07 Sep 2020 12:12:26 GMT

jloisel commented on pull request #616:
URL: https://github.com/apache/jmeter/pull/616#issuecomment-688285398


   Hi Philippe, Thanks for reviewing this pull request. Do you have any idea on how to fix
both the issue (do not escape remove escapes from escaped json, which relies on `\`) and support
escaping `\`? I may have missed something. 
   
   I thought about splitting the method to have both old and new behavior, depending on the
context.
   
   One thing is sure, replacing `\\` by `\` when the content contains a variable is causing
issues as it alters the content of the request. 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



Mime
View raw message