network.js: convert rpc protocol to lower case (#4855)

Fixes #4253
This commit is contained in:
pinkiebell 2018-07-25 00:03:36 +02:00 committed by Whymarrh Whitby
parent fed9ae0dee
commit 3667f3cb85
2 changed files with 4 additions and 0 deletions

View File

@ -7,6 +7,7 @@
- Add Trezor Support
- Allow to remove accounts (Imported and Hardware Wallets)
- [#4840](https://github.com/MetaMask/metamask-extension/pull/4840): Now shows notifications when transactions are completed.
- [#4855](https://github.com/MetaMask/metamask-extension/pull/4855): network.js: convert rpc protocol to lower case.
## 4.8.0 Thur Jun 14 2018

View File

@ -9,6 +9,7 @@ const extend = require('xtend')
const EthQuery = require('eth-query')
const createEventEmitterProxy = require('../../lib/events-proxy.js')
const log = require('loglevel')
const urlUtil = require('url')
const {
ROPSTEN,
RINKEBY,
@ -155,6 +156,8 @@ module.exports = class NetworkController extends EventEmitter {
}
_configureStandardProvider ({ rpcUrl }) {
// urlUtil handles malformed urls
rpcUrl = urlUtil.parse(rpcUrl).format()
const providerParams = extend(this._baseProviderParams, {
rpcUrl,
engineParams: {