Wrong return value for eth_requestAccounts

The documentation says the method eth_requestAccounts returns an array of accounts, however it returns a JSON-RPC response as of MetaMask 7.4.0. Probably a bug of MetaMask.

To reproduce

In any page containing an EthereumProvider for a browser installed MetaMask 7.4.0, evaluate the following statement:

ethereum.send('eth_requestAccounts').then(console.log)
// {id: 1, jsonrpc: "2.0", result: ['0x1234...']}

[danfinlay]

Yes, this would’ve been a breaking change, and so there was a very recent change to EIP-1193:

• The current send() behavior which returns Promise<{ results: <any> }> will remain.
• A new method request() will be added, which returns the intended result Promise<any>.

https://github.com/ethereum/EIPs/blob/master/EIPS/eip-1193.md

So this is indeed a problem with he docs: @BboyAkers is correct, and we should change to match his description.

[BboyAkers]

I’ll create a pull request and change it in the docs