Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[JENKINS-71567] Fix param name including & are not recognized #100

Merged
merged 1 commit into from
Jul 11, 2023

Conversation

Wadeck
Copy link
Contributor

@Wadeck Wadeck commented Jun 29, 2023

https://issues.jenkins.io/browse/JENKINS-71567
Encode the "param" value, otherwise name containing url special character can create issues.

Testing done

Manually tested with a parameter containing "&" and one without (to ensure non-regressions).

Submitter checklist

@basil basil added the bug Incorrect or flawed behavior label Jun 29, 2023
@basil basil requested a review from klimas7 June 29, 2023 20:12
@basil basil merged commit cb857a9 into jenkinsci:master Jul 11, 2023
@basil
Copy link
Member

basil commented Jul 11, 2023

Released in 0.9.19.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Incorrect or flawed behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants