search

statechannels / SCBridge-Wallet

MIT License
1 stars 2 forks source link

fix off-chain signature validation #112

Closed NiloCK closed 11 months ago

NiloCK commented 11 months ago

recoverSigner attempts are now skipped - some massaging of inputs should fix this.

geoknee commented 11 months ago

For example here https://github.com/statechannels/nitro-smart-contract-wallet/blob/c4ef557bbff3271d80c99155ba2f63484567aa0c/clients/OwnerClient.ts#L70

We probably want to write a test to exercise this logic and reproduce the bug, since getting there manually isn't straightforward on main just now.