-
Notifications
You must be signed in to change notification settings - Fork 22
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
[Bug]: Not always using unique onchain receive address #2078
Comments
Do you still have this issue? It might be fixed in #2210 |
Looks like the onchain wallet is not refreshed somewhere:
Simulator.Screen.Recording.-.iPhone.15.-.2024-10-02.at.12.23.11.mp4Then if you pull to refresh you'll get a new address. |
I haven't seen this recently with receive address, but we have seen change address reuse in testing session. |
@catch-21 can you test it again in order to see if Ivan need to spend time on it, please? |
Can easily reproduce this with a new wallet:
Simulator.Screen.Recording.-.iPhone.15.-.2024-11-07.at.15.40.21.mp4Feels like this should be handled in beignet but fixed here in Bitkit for now. |
Tested on 1.0.7 (139) 3b9a833 and (141) 97851ff using regtest wallet Screen-2024-12-03-100206.mp4... although, after tx confirm the address persists still: Screen-2024-12-03-101012.mp4In case it is in any way useful: bitkit_ldk_logs_1733221273874.zip |
I tried to reproduce this by receiving roughly 50 transactions. I got a unique address everytime. I didn't have to use pull-to-refresh. Tested on 041b824 Here's the list of addresses:
|
Looks like it is hard to reproduce on the regtest, might be easier on the mainnet |
I experienced this again today on mainnet. Steps:
|
Again today on mainnet with address bc1q5em7x9uf0qw88pwzn49cghfqxak9utd05fhv5u Similar to above, I closed a channel and it returned funds to an address that had already been used to return funds from spending balance the day before. |
Thing with closing channels is expected because you have to provide an address for channel refunds at the opening |
Sure. Maybe a dedicated derivation path to isolate those, a separate account? I would say extend the fourth level used for change but that would break BIP-44. |
Describe the bug
My wallet has not always been using a new unique address for each onchain receive. The latest transaction I took is f23b28ed8e8d4faf942e37824994818163cf4aa7017ea6592d4e3e88fd36d461 and that is to address bc1qqraqjlu9wgxsm8fsf4n8anhyzlkfap0f7eefca, which currently has 11 transactions over the past 3 months. When I go to receive again the same address is suggested.
This was once also the case for bc1q7t9a52ygc8yucya7zm0ypyu4ys3sgay9pq2ecc
Reproduce
Screenshots / Recording
Operating system
Android 13 TKQ1.221114.001
Bitkit version
v128
Log output
No response
The text was updated successfully, but these errors were encountered: