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

Loosen restriction on http(s) #134

Closed
mattgarrish opened this issue Oct 23, 2019 · 3 comments · Fixed by #135
Closed

Loosen restriction on http(s) #134

mattgarrish opened this issue Oct 23, 2019 · 3 comments · Fixed by #135

Comments

@mattgarrish
Copy link
Member

As noted by @BigBlueHat in #131 (comment), the restriction to using http(s) for resources when we're not bound to web publications is probably overly restrictive.

Reproducing the relevant parts of that thread below:

BigBlueHat

What about relative URLs?

I'm also concerned about restricting this to http and https. Imagine if this spec were older and restricted it to just http URLs...and then https came along...i.e. not sure restricting scheme--when we don't also restrict a protocol or intended API usage--makes much sense.

iherman
...
However, I fully agree with the second comment. It actually may also create issues with off-line UA implementations which may, in URI terms, rely on file: URIs, in fact, because everything is within a package. So yes, it is clearly an issue to discuss

@mattgarrish
Copy link
Member Author

For context, here's the current restriction:

Manifest URLs are restricted to only the http and https schemes [url].

https://w3c.github.io/pub-manifest/#value-url

@mattgarrish
Copy link
Member Author

I'm fine with dropping the restriction, fwiw, especially at this level. If profiles want/need to restrict the schemes, better to leave that decision to be made at the practical level.

@dauwhe
Copy link

dauwhe commented Oct 23, 2019

It's possible bundled exchanges would be a new protocol.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants