Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrong signature when signing message? #45

Closed
bonustrack opened this issue Sep 1, 2020 · 7 comments
Closed

Wrong signature when signing message? #45

bonustrack opened this issue Sep 1, 2020 · 7 comments

Comments

@bonustrack
Copy link

I'm using Walletlink in https://demo.snapshot.page to sign message but for some reason the signature created by is diffent than Metamask, i canot validate the signed message if it come from Walletlink. Any idea whats wrong? Do you have existing app using signing message ability?

@jurosh
Copy link

jurosh commented Oct 20, 2020

Same issue here... signature seems to be incorrect for some reason. Using Walletlink with ethers.js web3 provider...

I see that in repo are already more opened issues without any answer.. is Walletlink not maintained anymore ?

@petejkim
Copy link
Contributor

Hmmm are you using personal_sign?

@tsudmi
Copy link

tsudmi commented Mar 24, 2021

Would it be possible to switch to a standard signatures format?

@redben
Copy link

redben commented Apr 23, 2021

Same issue here... signature seems to be incorrect for some reason. Using Walletlink with ethers.js web3 provider...

I see that in repo are already more opened issues without any answer.. is Walletlink not maintained anymore ?

Setting provider.isMetaMask to true should fix it. Also redefine the property as it is not writable:
Object.defineProperties(walletconnectProvider, { isMetaMask: { value: true} })

@germanllop
Copy link

Same issue here... signature seems to be incorrect for some reason. Using Walletlink with ethers.js web3 provider...
I see that in repo are already more opened issues without any answer.. is Walletlink not maintained anymore ?

Setting provider.isMetaMask to true should fix it. Also redefine the property as it is not writable:
Object.defineProperties(walletconnectProvider, { isMetaMask: { value: true} })

This worked like a charm, can you explain what wrong with the other way please?

@redben
Copy link

redben commented Aug 10, 2021

@bangtoven
Copy link
Contributor

hi. sorry for the belated response. we are marking stale issues closed for now. please open a new one if needed. thanks.

bangtoven added a commit that referenced this issue Feb 29, 2024
* fix _connected check

* fix

* asf
bangtoven added a commit that referenced this issue Feb 29, 2024
* fix _connected check

* fix

* asf
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

7 participants