Drop origin key from requests for Parity compatibility

feature/default_network_editable
Dan Finlay 8 years ago
parent ae2f0f585f
commit e197e3808f
  1. 1
      CHANGELOG.md
  2. 5
      app/scripts/metamask-controller.js

@ -5,6 +5,7 @@
- Add a link to the transaction in history that goes to https://metamask.github.io/eth-tx-viz - Add a link to the transaction in history that goes to https://metamask.github.io/eth-tx-viz
too help visualize transactions and to where they are going. too help visualize transactions and to where they are going.
- Show "Buy Ether" button and warning on tx confirmation when sender balance is insufficient - Show "Buy Ether" button and warning on tx confirmation when sender balance is insufficient
- Change behavior that prevented Parity client compatibility, where we included a request site origin on rpc calls.
## 2.12.1 2016-09-14 ## 2.12.1 2016-09-14

@ -75,6 +75,10 @@ module.exports = class MetamaskController {
} }
onRpcRequest (stream, originDomain, request) { onRpcRequest (stream, originDomain, request) {
/* Commented out for Parity compliance
* Parity does not permit additional keys, like `origin`,
* and Infura is not currently filtering this key out.
var payloads = Array.isArray(request) ? request : [request] var payloads = Array.isArray(request) ? request : [request]
payloads.forEach(function (payload) { payloads.forEach(function (payload) {
// Append origin to rpc payload // Append origin to rpc payload
@ -86,6 +90,7 @@ module.exports = class MetamaskController {
payload.params.push({ origin: originDomain }) payload.params.push({ origin: originDomain })
} }
}) })
*/
// handle rpc request // handle rpc request
this.provider.sendAsync(request, function onPayloadHandled (err, response) { this.provider.sendAsync(request, function onPayloadHandled (err, response) {

Loading…
Cancel
Save