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.

Sign up for GitHub

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

Jump to bottom

XSS protection causes tags in issue explorer to be inproperly escaped #5387

Closed
kuhnchris opened this issue Oct 25, 2019 · 1 comment · Fixed by #5388
Closed

XSS protection causes tags in issue explorer to be inproperly escaped #5387

kuhnchris opened this issue Oct 25, 2019 · 1 comment · Fixed by #5388
Assignees
@danlipert
Labels
bug This is something that isn't working as intended. Gitcoin Issue Explorer Gitcoin Issue Explorer

Comments

@kuhnchris
Copy link
Contributor

Describe the bug
C#, C++ and anything else with "special characters" is getting stripped.

To Reproduce
dashboard->bounties->load more bounties->redirection to explorer

Expected behavior
C# and C++ being honored

Screenshots
grafik

grafik
@kuhnchris kuhnchris added Gitcoin Issue Explorer Gitcoin Issue Explorer security labels Oct 25, 2019
@thelostone-mc thelostone-mc mentioned this issue Oct 25, 2019
Merged
@danlipert danlipert added bug This is something that isn't working as intended. 2 labels Oct 28, 2019
@danlipert
Copy link
Contributor

danlipert commented Oct 28, 2019 •
edited
Loading

@frankchen07 we should get this in the next sprint EDIT: woops I just noticed Aditya already got it :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@danlipert danlipert

Labels
bug This is something that isn't working as intended. Gitcoin Issue Explorer Gitcoin Issue Explorer
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

bug: update scrub logic thelostone-mc/web
4 participants
@kuhnchris @danlipert @thelostone-mc @frankchen07