Skip to content

app: keep the wallet update task alive - #165

Open
octobocto wants to merge 1 commit into
LayerTwo-Labs:masterfrom
octobocto:sep-27-thunder-wallet-task
Open

octobocto wants to merge 1 commit into
LayerTwo-Labs:masterfrom
octobocto:sep-27-thunder-wallet-task

Conversation

@octobocto

@octobocto octobocto commented Sep 27, 2026 •

Copy link
Copy Markdown
Contributor

A node stopped reporting new coins until a restart: App derives Clone, and Drop for App aborted the shared wallet update task, so the first dropped clone killed it for every other clone. The create_deposit, mine, connect_block and get_block_template handlers each clone the app per request, which is why a node that mines its own blocks still looked correct. Drop now aborts only when Arc::get_mut gives the last reference, the way NetTaskHandle and MainchainTaskHandle already do.

@octobocto
octobocto force-pushed the sep-27-thunder-wallet-task branch from c744ae4 to 5a855bd Compare September 27, 2026 14:59
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

Successfully merging this pull request may close these issues.

1 participant