-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
Fix for Cross Site Scripting Rule vulnerabilities #24029
base: master
Are you sure you want to change the base?
Conversation
04362c8
to
8104d78
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Release note:
* Fix provided for Cross Site Scripting Rule vulnerabilities. :pr:
24029--->
* Fix Cross Site Scripting Rule vulnerabilities. :pr:24029
presto-main/src/main/java/com/facebook/presto/server/AsyncPageTransportServlet.java
Outdated
Show resolved
Hide resolved
|
3b84318
to
acb2def
Compare
Minor nits for the release note entry.
|
acb2def
to
0733e81
Compare
9868ca2
to
60d8810
Compare
presto-main/src/main/java/com/facebook/presto/server/AsyncPageTransportServlet.java
Outdated
Show resolved
Hide resolved
Thanks for the release note entry! A few formatting notes to follow the Release Note Guidelines.
|
86a2f97
to
4f5f268
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM % nit
presto-main/src/main/java/com/facebook/presto/server/AsyncPageTransportServlet.java
Outdated
Show resolved
Hide resolved
6605595
to
dd1ebdc
Compare
83e437f
to
c539f90
Compare
c539f90
to
043e0e2
Compare
Description
This PR addressing a Cross-Site Scripting (XSS) vulnerability discovered during a static scan.
Motivation and Context
Fixing XSS vulnerabilities is critical to securing web applications and protecting both user data and website integrity.
Impact
Test Plan
Contributor checklist
Release Notes
Please follow release notes guidelines and fill in the release notes below.