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

Deprecate Web Browser XSS Protection Not Enabled? #5849

Closed
kingthorin opened this issue Feb 6, 2020 · 3 comments · Fixed by zaproxy/zap-extensions#2297
Closed

Deprecate Web Browser XSS Protection Not Enabled? #5849

kingthorin opened this issue Feb 6, 2020 · 3 comments · Fixed by zaproxy/zap-extensions#2297
Assignees

Comments

@kingthorin
Copy link
Member

kingthorin commented Feb 6, 2020

See: https://security.stackexchange.com/questions/225415/are-favicon-ico-robots-text-sitemap-xml-vulnerable-to-xss

Per:
https://caniuse.com/#search=X-XSS-Protection
https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/X-XSS-Protection


image

@psiinon
Copy link
Member

psiinon commented Feb 6, 2020

Yeah, I noticed that too. I'm good with deprecating it.

@kingthorin
Copy link
Member Author

kingthorin commented Feb 6, 2020

Ref:

  • Addon: pscan release
  • Rule: HeaderXssProtectionScanner.java
  • PluginID: 10016

@kingthorin kingthorin self-assigned this Feb 6, 2020
kingthorin added a commit to kingthorin/zaproxy that referenced this issue Feb 7, 2020
kingthorin added a commit to kingthorin/zap-extensions that referenced this issue Feb 7, 2020
kingthorin added a commit to kingthorin/zap-extensions that referenced this issue Feb 7, 2020
kingthorin added a commit to kingthorin/zap-extensions that referenced this issue Feb 7, 2020
@lock
Copy link

lock bot commented May 20, 2020

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked and limited conversation to collaborators May 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging a pull request may close this issue.

2 participants