fix: remove condition to allow 'cosmos' as valid value for verifyingContract field in typed signatures

MetaMask / eth-json-rpc-middleware

Ethereum middleware for composing an Ethereum provider using json-rpc-engine. Intended to replace provider-engine

ISC License

179 stars 88 forks source link

fix: remove condition to allow 'cosmos' as valid value for verifyingContract field in typed signatures #337

Open jpuri opened 2 months ago

jpuri commented 2 months ago

We have added code to allow cosmos as valid verifyingContract value for typed signatures. The reason for this is to keep support signature signing on cosmos network.

A lot of cosmos dapps use this deprecated sdk with hard coded value of verifyingContract: https://github.com/evmos/ethermint/blob/main/ethereum/eip712/domain.go#L29

Once users stop using this SDK we can remove this hard coded condition.

Ref: https://github.com/MetaMask/metamask-extension/issues/26980

legobeat commented 2 months ago

Perhaps contract verification can be toggled/configured via a constructor parameter in a way that could make this transition non-breaking?