jenkinsci / peass-ci-plugin

Jenkins plugin for peass to support performance measurement in CI
https://plugins.jenkins.io/peass-ci/
GNU Affero General Public License v3.0
4 stars 12 forks source link

Prevent "${" from being interpreted by JavaScript #207

Closed mawHBT closed 1 year ago

mawHBT commented 1 year ago

What feature do you want to see added?

If sourcecode contains ${, e.g. like in @KafkaListener(topics = "${kafka.chargingStationEventTopic}"...), sourcecode is not shown in action for a specific test. ${ is JavaScript-syntax, which probably causes the error.

Upstream changes

No response

DaGeRe commented 1 year ago

Thanks for the report. This should be fixed by https://github.com/DaGeRe/peass/commit/4f0c0b61e5fb7b4d6c95eae61f28dcbc75e20a01 - can you test it?

mawHBT commented 1 year ago

This seems to work now.