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

Add IPFS and IPNS as safelisted schemes for registerProtocolHandler() #7911

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

javifernandez
Copy link
Contributor

@javifernandez javifernandez commented May 10, 2022

This commit updates the list of the safelisted schames that can
be handled by the registerProtocolHandler method to include IPFS
and IPNS schemes.

It also touches the current list to remove the bitcoin documentation
link, since it's already referenced on IANA's page for URI schemes.

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


/system-state.html ( diff )

This commit updates the list of the safelisted schames that can
be handled by the registerProtocolHandler method to include IPFS
and IPNS schemes.

It also touches the current list to remove the bitcoin documentation
link, since it's already referenced on IANA's page for URI schemes.
@javifernandez
Copy link
Contributor Author

With this PR I'm trying to unblock the discussion ongoing in the PR#5482 where we are trying to add several dweb protocols to the scheme safelist. During the discussion in the Firefox standard-position request, it was suggested to split up the PR to analyze each protocol independently, to address the concern of endorsing the technologies behind the schemes.

I guess we can assume Chrome's support given that a CL has already landed upstream with all these schemes safelisted.

Regarding Firefox, as mentioned before, there is an ongoing discussion in the standatd-postiion request submitted for the PR#5482, but perhaps we would need a new one specifically about IPFS and IPNS, as suggested by @dbaron

There is already a tentative test in the WPT repository, with all the protocols. We can merge it with the test defined for the current safelist schemes once this PR is merged.

@javifernandez
Copy link
Contributor Author

cc @fred-wang who was the author of the original PR#5482,

@ben221199
Copy link

Why this isn't merged yet? 😕

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

Successfully merging this pull request may close these issues.

2 participants