whatwg / url

URL Standard
https://url.spec.whatwg.org/
Other
532 stars 140 forks source link

Return null origin of "blob:" URL containing inner non-"http(s):" URL #771

Closed rmisev closed 1 year ago

rmisev commented 1 year ago

Fixes: #770.

(See WHATWG Working Mode: Changes for more details.)


Preview | Diff

annevk commented 1 year ago

@ricea @zcorpan thoughts?

zcorpan commented 1 year ago

This seems OK to me. cc @valenting

valenting commented 1 year ago

This makes sense to me. I filed Bug 1834703 - Return null origin of "blob:" URL containing inner "blob:" URL for Firefox changes. Thank you for the tests.

annevk commented 1 year ago

@ricea to be clear, all browsers have failures here. E.g., "Origin parsing: <blob:file://host/path> without base" yields !EQ("null", "file://") in Chromium. (Same for ftp, ws, and wss.)

@rmisev do you want to file the remaining implementation bugs?

ricea commented 1 year ago

@ricea to be clear, all browsers have failures here. E.g., "Origin parsing: blob:file://host/path without base" yields !EQ("null", "file://") in Chromium. (Same for ftp, ws, and wss.)

I am okay with that.

rmisev commented 1 year ago

@rmisev do you want to file the remaining implementation bugs?

@annevk, I think you would fill in the bug reports faster than me.