AlphaWallet / AlphaWallet/alpha-wallet-android

WalletConnect v2: Signing from a different wallet

オープン
#3,185 コメント 0 件 リアクション 0 件 担当者 1 名 @seabornlee が担当を希望しています GitHub で見る
主要言語
Java
スター
646
フォーク
579
PR マージ指標
30日以内にマージされた PR はありません

説明

With WallectConnect v2, if you sign from a different wallet the signature comes from the currently selected wallet.

Reproduce:

go to the react app: https://react-app.walletconnect.com/

Connect using your "wallet A".

Now change wallets to your (valid) "wallet B".

Click 'personal_sign' and process the transaction, which fails (because the app is expecting a signature from wallet A, but you provide a sig from Wallet B).

I see that the address is marked with a warning, but there should be one of the following things:

1. Refuse to sign and state why (Not a good solution).
2. Don't sign and offer to change user keys (Not good either).
3. Before popping open the Sign Dialog (after user clicks next), pop up a warning which is similar to the one you use in the 'Token Shortcut' addition - the user can continue to sign with the original key, or they can abort signing. Use the proper wallet in the sign dialog (NB ActionSheetSignDialog has its own ViewModel with its own wallet fetch).

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。