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

The origin of "blob:" URL containing inner "blob:" URL #770

Closed
rmisev opened this issue May 8, 2023 · 1 comment · Fixed by #771
Closed

The origin of "blob:" URL containing inner "blob:" URL #770

rmisev opened this issue May 8, 2023 · 1 comment · Fixed by #771
Labels
topic: model For issues with the abstract-but-normative bits

Comments

@rmisev
Copy link
Member

rmisev commented May 8, 2023

Different browsers return different origin values for such URLs.
For example, the results of parsing blob:blob:http://host:123/path are:

Browser or standard origin
URL standard http://host:123
Chromium null
Firefox http://host:123
Safari blob://

I would prefer Chromium's behavior, even though it differs from the URL standard. There are two reasons for this:

  1. Using the recursive origin's algorithm of the URL standard, it is easy to cause out of memory error by, for example, providing a URL with millions of blob: in it: blob:blob:blob: ... blob:http://host:123/path
  2. Maybe I am wrong, but I think that a URL with many embedded blob: has no practical use.

If you agree, I can prepare PR and tests.

@annevk annevk added the topic: model For issues with the abstract-but-normative bits label May 8, 2023
@annevk
Copy link
Member

annevk commented May 8, 2023

I think you are correct. In fact, we should probably only do recursion when the scheme is an HTTP(S) scheme.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: model For issues with the abstract-but-normative bits
Development

Successfully merging a pull request may close this issue.

2 participants