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

Tests for history.pushState() URL rewriting #30182

Merged
merged 3 commits into from
Sep 9, 2021

Conversation

domenic
Copy link
Member

@domenic domenic commented Aug 25, 2021

@domenic
Copy link
Member Author

domenic commented Sep 8, 2021

Summary of results for the scheme-based version versus current browsers:

  • Firefox still gets the wrong exception a lot
  • Firefox disallows rewriting blob URLs, but should allow rewriting them to themselves or hashed versions of themselves
  • Firefox disallows rewriting about: URLs, but should allow rewriting them to themselves or hashed versions of themselves
  • Firefox disallows rewriting data: URLs, but should allow rewriting them to themselves or hashed versions of themselves
  • Chrome allows rewriting blob: URLs to new same-origin search or path, but should not
  • Chrome allows rewriting data: URLs to a new search, but should not
  • Chrome disallows rewriting about: URLs, but should allow rewriting them to themselves or hashed versions of themselves
  • Chrome disallows rewriting the path of sandboxed URLs, but should allow it
  • Safari allows rewriting blob: URLs to new same-origin search or path, but should not
  • Safari allows rewriting data: URLs to a new search, but should not
  • Safari disallows rewriting the path of sandboxed URLs, but should allow it

@domenic domenic merged commit 130d57f into master Sep 9, 2021
@domenic domenic deleted the pushstate-replacestate-url-rewriting branch September 9, 2021 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants