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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ethereum url scheme to safelist #4287

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

Conversation

brunobar79
Copy link

@brunobar79 brunobar79 commented Jan 13, 2019

Added ethereum: url scheme to the safelist protocol handler.

It has the same functionality as the bitcoin: protocol in terms of payments and transactions, but in the mid-long term it could be used to identify 脨Apps

For more information about Ethereum you can check out https://www.ethereum.org/


馃挜 Error: Wattsi server error 馃挜

PR Preview failed to build. (Last tried on Jan 15, 2021, 7:59 AM UTC).

More

PR Preview relies on a number of web services to run. There seems to be an issue with the following one:

馃毃 Wattsi Server - Wattsi Server is the web service used to build the WHATWG HTML spec.

馃敆 Related URL

<html>
<head><title>504 Gateway Time-out</title></head>
<body bgcolor="white">
<center><h1>504 Gateway Time-out</h1></center>
<hr><center>nginx/1.10.3</center>
</body>
</html>

If you don't have enough information above to solve the error by yourself (or to understand to which web service the error is related to, if any), please file an issue.

@domenic
Copy link
Member

domenic commented Jan 24, 2019

This is probably blocked on #3998. See all the attempts at expanding this list in e.g. #1829, #2546, #3935.

@fred-wang
Copy link
Contributor

Discussion about IANA registration moved to ethereum/consensus-specs#1808

@fred-wang
Copy link
Contributor

@brunobar79 Thank you for the report and pull request. I've included ethereum in #5482 as part of a more general update of decentralized schemes.

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

Successfully merging this pull request may close these issues.

None yet

3 participants