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

Increase CLOSED timeout from 10s to 1min #2281

Merged
merged 1 commit into from
May 24, 2022
Merged

Increase CLOSED timeout from 10s to 1min #2281

merged 1 commit into from
May 24, 2022

Conversation

pm47
Copy link
Member

@pm47 pm47 commented May 23, 2022

This gives more time to rollback the funding tx in case where we moved from WAIT_FOR_FUNDING_INTERNAL->CLOSED and unlock utxos.

We could do it differently, with an additional state, or by only sending the "shutdown" symbol after we receive the wallet response for this specific transition, but it would be more complicated.

This gives more time to rollback the funding tx in case where we moved
from `WAIT_FOR_FUNDING_INTERNAL`->`CLOSED`.

We could do it differently, with an additional step, or by only sending
the "shutdown" symbol after we receive the wallet response for this
specific transition, but it would be more complicated.
@pm47 pm47 changed the title Increase closed timeout from 10s to 1min Increase CLOSED timeout from 10s to 1min May 23, 2022
@pm47 pm47 merged commit 9c7ddcd into master May 24, 2022
@pm47 pm47 deleted the closed-timeout branch May 24, 2022 08:09
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.

2 participants