[GitHub] [jmeter] pmouawad commented on pull request #727: Bug 66269: Add NoThreadClone to HeaderManager to reduce heap utilization

["pmouawad (via GitHub)" <gi...@apache.org>]

pmouawad commented on PR #727:
URL: https://github.com/apache/jmeter/pull/727#issuecomment-1543558928

   > The change would impact the ones who attempt to add/remove headers dynamically via JSR223 or something similar.
   > 
   > For instance, https://sqa.stackexchange.com/questions/41708/jmeter-header-manager-alters-the-headers-during-test attempts
   > 
   > ```java
   > sampler.getHeaderManager().add(new Header("Authorization","Bearer " + vars.get("BEARER"))); 
   > ```
   
   Hello @vlsi , how would people requiring this feature do then ?
   In my performance testing experience I know that for example to be able to test JDEdwards (and another custom product), you need similar code. So if it's not more possible, it's an important problem.
   
   Thanks


-- 
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.

To unsubscribe, e-mail: dev-unsubscribe@jmeter.apache.org

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