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

"Is it my address" doesn't work for offline device #6232

Open
TimYorke opened this issue Mar 4, 2024 · 5 comments
Open

"Is it my address" doesn't work for offline device #6232

TimYorke opened this issue Mar 4, 2024 · 5 comments
Assignees

Comments

@TimYorke
Copy link

TimYorke commented Mar 4, 2024

BlueWallet 6.5.5

I've discovered that the "Is it my address" feature doesn't recognise the receive address of an imported wallet (BlueWallet Vault) until I've connected BlueWallet to an electrum server and downloaded the transactions.

Before connecting to the electrum server, it doesn't recognise the scanned address, but afterwards and all the transactions have been downloaded, it correctly identifies the wallet when scanning the same address as before.

This is a problem when you want to use BlueWallet as a cold, air-gapped signing device. It's important to be able to check the receive address on a cold device. A side effect is that it displays incorrect amounts on transactions you're signing (because I guess it isn't subtracting the change UTXO).

I think a workaround will be to import the wallet without any seed phrases, connect and download transactions, and then permanently disconnect networks and then enter the seed phrase.

@limpbrains
Copy link
Collaborator

Hi! Thanks for reporting. Do you know the index of the address you are trying to find in BW?

@ricaum
Copy link

ricaum commented Mar 8, 2024

Parece que há um problema no layout, quando vai recuperar a seed no modo off-line. Aparece um gif girando (loding) e a opção (Use um caminho de derivação personalizada) vai para baixo e a opção fica escondida.

@limpbrains
Copy link
Collaborator

@ricaum I think your issue is unrelated to the one described above. Please open new issue, describe how to reproduce it, attach a video if possible, thx

@limpbrains
Copy link
Collaborator

I believe the issue lies in the offline wallet being unaware of the number of addresses already used. By default, BW generates only 20 addresses for receiving and 20 for change (gap limit). We need to update these indexes for offline wallets, but the exact location for this update is uncertain. Simply creating and signing a PSBT does not guarantee its broadcast. Perhaps we should only broadcast it if the user exports it using a file or QR code. Alternatively, we could introduce another index, such as "max_saw_address_index," update it in wallet.createTransaction, and consider it when generating the address list.

Any thoughts on this, @Overtorment?

@Overtorment
Copy link
Member

It's even less for multisig, 10 i think. So blank just-imported multisig will be aware of only 10 receive addresses. How is it supposed to know that it was heavily used outside?

However, when importing psbts, we could move the last used index, as iirc psbts carry paths for used inputs and we could get the exact index

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

No branches or pull requests

4 participants