kenchris / urlpattern-polyfill

URLPattern polyfill
https://www.npmjs.com/package/urlpattern-polyfill
MIT License
268 stars 31 forks source link

default options should be used to parse cannot-be-a-base-URL pathnames #30

Closed wanderview closed 3 years ago

wanderview commented 3 years ago

Currently we are treating / characters specially in all pathnames, but this only makes sense for standard URLs. Things like data: URLs don't structure around slashes in the pathname. We should therefore be passing default path-to-regexp options when parsing these cannot-be-a-base-URL pathnames.

See also:

https://bugs.chromium.org/p/chromium/issues/detail?id=1234865