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

Search input: sanitize input #418

Merged
merged 1 commit into from
Oct 6, 2021
Merged

Conversation

eruizgar91
Copy link

@eruizgar91 eruizgar91 commented Oct 6, 2021

Description

Fix xss vulnerability

Other changes

Fix from upstream blockscout#4713

Tested

An explanation of how the changes were tested or an explanation as to why they don't need to be.
Add any artifacts (links, screenshots) you can include to increase the reviewers' confidence in the change.

Issues

Backwards compatibility

Brief explanation of why these changes are/are not backwards compatible.

Checklist

  • If I added new functionality, I added tests covering it.
  • If I fixed a bug, I added a regression test to prevent the bug from silently reappearing again.
  • I added code comments for anything non trivial.
  • I added documentation for my changes.
  • If I added/changed/removed ENV var, I submitted a PR to https://github.com/celo-org/monorepo to update the list and default values of env vars.
  • If I add new indices into DB, I checked, that they are not redundant with PGHero or other tools.

@eruizgar91 eruizgar91 requested a review from a team October 6, 2021 08:39
@github-actions
Copy link

github-actions bot commented Oct 6, 2021

Unit Test Results

       3 files  ±0     222 suites  ±0   3m 7s ⏱️ ±0s
2 048 tests ±0  2 003 ✔️ ±0  45 💤 ±0  0 ±0 

Results for commit 8484e47. ± Comparison against base commit 8484e47.

♻️ This comment has been updated with latest results.

@eruizgar91 eruizgar91 merged commit 8484e47 into master Oct 6, 2021
@eruizgar91 eruizgar91 deleted the bugfix/xss-vulnerability branch October 6, 2021 10:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants