fix(server): Removes minor XSS vulnerability. Fixes #3942 #3975

alexec · 2020-09-09T15:24:23Z

Checklist:

Either (a) I've created an enhancement proposal and discussed it with the community, (b) this is a bug fix, or (c) this is a chore.
The title of the PR is (a) conventional, (b) states what changed, and (c) suffixes the related issues number. E.g. "fix(controller): Updates such and such. Fixes #1234".
I've signed the CLA.
~~I have written unit and/or e2e tests for my change. PRs without these are unlikely to be merged.~~
My builds are green. Try syncing with master if they are not.
My organization is added to USERS.md.

fix(server): Remove XSS vulnerability. Fixes argoproj#3942

1a1ccea

alexec added type/bug type/security Security related labels Sep 9, 2020

alexec marked this pull request as ready for review September 9, 2020 15:46

alexec changed the title ~~fix(server): Remove XSS vulnerability. Fixes #3942~~ fix(server): Removes minor XSS vulnerability. Fixes #3942 Sep 9, 2020

alexec assigned simster7 Sep 10, 2020

simster7 approved these changes Sep 10, 2020

View reviewed changes

alexec merged commit 7b1d17a into argoproj:master Sep 10, 2020

alexec deleted the state branch September 10, 2020 19:25

alexec added a commit that referenced this pull request Sep 10, 2020

fix(server): Remove XSS vulnerability. Fixes #3942 (#3975)

089e186

alexec added a commit that referenced this pull request Sep 10, 2020

fix(server): Remove XSS vulnerability. Fixes #3942 (#3975)

a8ba447

alexec added this to the v2.10 milestone Sep 10, 2020

Provide feedback