Consider using the basic URL parser (i.e. disallowing blob URLs) about web-share HOT 2 CLOSED

I don't think it matters either way. Because I am not parsing a URL string to a URL record, then transmitting the URL record (which would have a blob in the URL parser case). I am serializing the URL record back to a URL string again. So using URL Parser vs Basic URL Parser should be equivalent (blob URLs are implicitly allowed, but we would not transmit the blob object).

URL Parser seemed a bit higher level which is why I went with it. Should I still change to Basic URL Parser?

If you want to disallow blob URLs, I think we need an explicit check for it and throw a TypeError.

from web-share.

You're right on both counts; the parser used doesn't matter in this case, and my suggestion doesn't properly disallow blobs.

Probably leaving things scheme-agnostic makes more sense then, if there's no natural machinery for excluding blobs. So yeah, I'll close; thanks for setting me straight!

from web-share.