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

Adding NEAR EVM #110

Merged
merged 2 commits into from
Oct 7, 2020
Merged

Adding NEAR EVM #110

merged 2 commits into from
Oct 7, 2020

Conversation

ilblackdragon
Copy link
Contributor

NEAR EVM behaves like Ethereum shard on NEAR and support set of tools like Truffle / Metamask, hence the chain-id requirement.

Details about NEAR EVM can be found in WIP spec: near/NEPs#106

The id for MainNet is hex("NEAR")

@ligi
Copy link
Member

ligi commented Oct 5, 2020

Thanks for the PR - are there really no faucets for the testnet?

@ilblackdragon
Copy link
Contributor Author

ilblackdragon commented Oct 6, 2020

Good point, added faucet for Testnet.

@ligi ligi merged commit 784e64b into ethereum-lists:master Oct 7, 2020
@ilblackdragon ilblackdragon deleted the add-near-evm branch October 9, 2020 12:30
@mikedotexe
Copy link

@ligi a team member at NEAR pointed out that the "Short Name" is the same for both NEAR testnet and NEAR mainnet, will this cause issues?

@saurik
Copy link
Contributor

saurik commented Nov 30, 2020

Is this really an Ethereum-compatible chain? I was excited to see it added, but the RPC endpoint doesn't seem to respond to any Ethereum methods. I've seen endpoints that don't support eth_chainId before (Cloudflare, for example) and I was annoyed but not too put off by eth_gasPrice not working... but, when eth_blockNumber didn't even work, I was left scratching my head as to what the listed RPC endpoint is accomplishing :(. Has anyone found any methods that I can expect to work? ;P

@ilblackdragon
Copy link
Contributor Author

@saurik No, RPCs are not Ethereum compatible.

There is a proxy web3 Provider here - https://github.com/near/near-web3-provider There will be also RPC proxy, later - there is an open bounty for building it.

Also the work is still happening to nail down the protocol update of adding EVM here - near/nearcore#3257, before it's going to make to TestNet / MainNet.

This PR was done ahead of time, so we can start testing tooling, like MetaMask and other tools to recognize NEAR.

@saurik
Copy link
Contributor

saurik commented Nov 30, 2020

@ilblackdragon If the RPC endpoint isn't Ethereum compatible then I'd argue it really shouldn't be listed in the rpc field of this Ethereum chain database, as it isn't actually usable or compatible with any of the other usages of the field. The expectation should be that I can, given just a chain's ID, pull its RPC endpoint from this database and talk to it (and to that end one of the things I've been working on a bit is automated testing of all the RPC endpoints in this database by doing a quick minimal check that, at least, calling eth_chainId returns the expected chain, which is important as sometimes these networks--I'm looking at you, Avalanche ;P--messes with their chains a bit while lauching).

@ilblackdragon
Copy link
Contributor Author

@saurik Good point, let me send a PR to remove RPC endpoints for now.

@pedrouid
Copy link
Collaborator

btw @ilblackdragon you should also consider introducing a CAIP for Near... here is an example of a current PR for EOSIO namespace (ChainAgnostic/CAIPs#33)

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 this pull request may close these issues.

5 participants