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

BlueWallet addresses may be bech32 in all uppercase. #665

Open
jdlcdl opened this issue Jan 19, 2025 · 3 comments · May be fixed by #667 or #666
Open

BlueWallet addresses may be bech32 in all uppercase. #665

jdlcdl opened this issue Jan 19, 2025 · 3 comments · May be fixed by #667 or #666

Comments

@jdlcdl
Copy link

jdlcdl commented Jan 19, 2025

Yet seen before tonight, a user in the main telegram group reported that seedsigner was not able to verify addresses from their BlueWallet native segwit wallet. Seedsigner was instead looking for a match on a legacy address that starts with 1.

After discussing, this was seen by BlueWallet v7.0.7 iOS (build 1736967217 master)

The qr decoded to something like:
"bitcoin:BC1Q..."
and seedsigner was searching for an address starting with 1, as if it had chopped "bitcoin:BC" from the front of the decoded string.

As soon as I lowercased that string and gave another qrcode to the user, seedsigner matched the address quickly.

This feels buggy on BlueWallet's side... because while uppercase can work with bech32 to make a more efficient qrcode, it doesn't work well with "bitcoin:" in lowercase.

Still, I suspect that seedsigner 1) should not be chopping anything off after "bitcoin:", and 2) it might be more tolerant when it can be sure that the entire address appears to be from the bech32 alphabet.

This looks recent at BlueWallet: Expect more issues.
BlueWallet/BlueWallet#7461

@dax-hub615

This comment has been minimized.

@jdlcdl
Copy link
Author

jdlcdl commented Jan 19, 2025

DISREGARD THE ABOVE POST. DO NOT FOLLOW ANY LINKS OUTSIDE OF THIS REPO. ALL PROBS CAN BE RESOLVED PUBLICLY AND SEEDSIGNER DEVS WILL NEVER TEMP YOU TO GO PRIVATE LIKE THE SCAMMER ABOVE!!!

@jdlcdl jdlcdl linked a pull request Jan 20, 2025 that will close this issue
12 tasks
@kdmukai
Copy link
Contributor

kdmukai commented Jan 20, 2025

Can confirm that BlueWallet Android v7.0.7 encodes segwit addresses in QR codes with all-caps. But the case-sensitive nested segwit and legacy address formats retain their mixed case / case sensitive chars in QR codes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants