-
Notifications
You must be signed in to change notification settings - Fork 31
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
javascript:
bypass via <svg>
and use
.
#84
Comments
Thanks! |
Seconded. The spec still pretends that SVG doesn't exist, which opens up this hole. The specific vector here seems to be |
Regression tests for: - WICG/sanitizer-api#84 - WICG/sanitizer-api#85 - WICG/sanitizer-api#86 Bug: 1203238 Change-Id: I844185c4b3295712f11d57fd0bf91efb9402823f
Regression tests for: - WICG/sanitizer-api#84 - WICG/sanitizer-api#85 - WICG/sanitizer-api#86 Bug: 1203238 Change-Id: I844185c4b3295712f11d57fd0bf91efb9402823f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2854853 Commit-Queue: Daniel Vogelheim <[email protected]> Reviewed-by: Yifan Luo <[email protected]> Cr-Commit-Position: refs/heads/master@{#877446}
Meanwhile, the spec knows SVG exists, but we haven't agreed on a "baseline" config for it yet. We should close this bug once that is there. If it matters, Chromium has a for-testing configuration supporting SVG & friends, which does not have |
Regression tests for: - WICG/sanitizer-api#84 - WICG/sanitizer-api#85 - WICG/sanitizer-api#86 Bug: 1203238 Change-Id: I844185c4b3295712f11d57fd0bf91efb9402823f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2854853 Commit-Queue: Daniel Vogelheim <[email protected]> Reviewed-by: Yifan Luo <[email protected]> Cr-Commit-Position: refs/heads/master@{#877446} NOKEYCHECK=True GitOrigin-RevId: 4aee5e3d88ce4742d367f7525db00ff4f851846b
Do all implementations block (See also #103.) |
with |
What was the conclusion here and why? We should have something before we close an issue. |
OK. |
A friendly, clever person submitted the following as a bypass of the
javascript:
URL handling in https://wicg.github.io/sanitizer-api/#handle-funky-elements:The text was updated successfully, but these errors were encountered: