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

Faling connection to dapp (CoinMarketCap) #21939

Closed
clauxx opened this issue Jan 16, 2025 · 2 comments · Fixed by #22026
Closed

Faling connection to dapp (CoinMarketCap) #21939

clauxx opened this issue Jan 16, 2025 · 2 comments · Fixed by #22026
Assignees
Labels
dapp-interactions wallet-core Issues for mobile wallet team
Milestone

Comments

@clauxx
Copy link
Member

clauxx commented Jan 16, 2025

Bug Report

Problem

When attempting to connect to https://coinmarketcap.com pairing succeeds, but it fails on the next step during authentication via personal_sign. Also the message is not visible to the user on the signing modal.

It seems like the issue is that most dapps send the personal_sign data as hex already, while here they send it as UTF-8. We should check if the data is sent as hex before hashing it, and if it's not - hex it ourselves. The same with displaying it - only decode it to UTF-8 if it is hex encoded.

Expected behavior

Can see the authentication message (message signing) and connect to CoinMarketCap successfully.

Actual behavior

Can't see the message nor connect to CoinMarketCap

Reproduction

  1. Go to https://coinmarketcap.com
  2. Press Connect
  3. Open Status app and log in
  4. Scan the QR code
  5. Approve pairing
  6. The dapp should react and prompt a captcha puzzle
  7. When done, it should send a personal_sign request
  8. The request message will be empty (!)
  9. After approving signing via password/biometrics/keycard, it should show an error (!)

Additional Information

  • Status version: nightly
  • Operating System: Android, iOS
@clauxx clauxx added dapp-interactions wallet-core Issues for mobile wallet team labels Jan 16, 2025
@clauxx clauxx modified the milestone: 2.33.0 Jan 16, 2025
@shivekkhurana
Copy link
Contributor

Did you check if this issue exists on Rainbow or Coinbase wallet too ?

@status-im status-im deleted a comment Jan 16, 2025
@clauxx clauxx added this to the 2.33.0 milestone Jan 16, 2025
@mohsen-ghafouri mohsen-ghafouri self-assigned this Feb 3, 2025
@mohsen-ghafouri
Copy link
Contributor

@shivekkhurana I just tested with Status Desktop version and it works fine, it successfully paired with CoinMarketCap. JFYI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dapp-interactions wallet-core Issues for mobile wallet team
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants