extension icon indicating copy to clipboard operation
extension copied to clipboard

Connecting on console.xyz sometimes errors

Open 314159265359879 opened this issue 6 months ago • 2 comments

I ran some tests but haven't been able to reproduce this. Tried signing in with another account than the active account. Tried signing in with Ledger (doesn't accept payload, known issue unrelated).

From Console: We see a fairly common error with Leather whereby they get a signature signing error when joining Console. It's mostly reported with Leather, but I assume it might have to do with Stacks.js because I've also seen it with Xverse. Typically clearing cache and trying again solves the issue.

We'd love to keep supporting Leather login, but would you be able to help us understand how to reduce this error for our users? We're a bit stuck.

image

I saw error 0021 once (instead of 0012 above) after signing in with two accounts then signing out on the dapp and connecting again (active account 1), and tried signing in with account 3... I see this in the console image image

Then singing in again, there is no issue. Connected to account 3.

I try to reproduce

  1. visit app.console.xyz
  2. Ctrl+F5
  3. Switch Leather extension to account 1
  4. Connect Leather on console dapp
  5. Select account 1
  6. Sign message, sign
  7. Go to profile, then settings
  8. Then click wallets
  9. Connect another leather account
  10. Now connect account 2 (active account in leather still set to account 1)
  11. See it appear...
  12. Close global settings popover
  13. Click profile upper right and sign out
  14. Connect Leather again
  15. Select account 3
  16. but I only saw the error once... not sure what caused it

Did some connecting of wallets, 2 is connected, I try to connect account 1 and I see this, but it isn't connected yet... image

314159265359879 avatar Feb 02 '24 23:02 314159265359879

Is this with Ledger or software mode?

markmhendrickson avatar Feb 08 '24 12:02 markmhendrickson

I spoke with the Console devs earlier. They'll try replacing the Stacks Connect methods with our RPC methods and report back.

kyranjamie avatar Feb 08 '24 12:02 kyranjamie

Confirmed by Console as fixed with Leather

markmhendrickson avatar Feb 26 '24 14:02 markmhendrickson

Reopening since this is apparently not fixed yet after all

markmhendrickson avatar Mar 25 '24 16:03 markmhendrickson

@kyranjamie Hi! This issue doesn't seem to be resolved. Would you be able to look at our code? Is it possible we are implementing something incorrect, or that the bug is on the Stacks side? Unclear. But we get a few of these errors every week. Thanks!

castig avatar Mar 25 '24 16:03 castig

Happy to audit implementation. Did you migrate to the RPC getAddresses method?

kyranjamie avatar Mar 25 '24 16:03 kyranjamie

I'm closing out as stale. Please reopen as needed.

markmhendrickson avatar Jul 15 '24 13:07 markmhendrickson