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

exposed a method to cancel all outstanding test plugins #273

Conversation

renepickhardt
Copy link
Contributor

especially uncertain about line 240
because I don't know what actually happens on a protocol level if one tries to cancel an htlc that has already been canceled. would be bad if the channel gets failed. it wasn't clearly to me specified at: https://github.com/lightningnetwork/lightning-rfc/blob/master/02-peer-protocol.md#removing-an-htlc-update_fulfill_htlc-update_fail_htlc-and-update_fail_malformed_htlc

if that was a problem anyway we would still have race condition problems if the call comes roughly at the same time when the timeout timer triggers anyway

@renepickhardt
Copy link
Contributor Author

this method does not work! in the current form as it triggers this exception:

INFO    plugin-paytest.py:   File \"/home/rpickhardt/lnplugins/plugins/paytest/paytest.py\", line 245, in rejectpaytests
INFO    plugin-paytest.py:     for secret in plugin.pending.keys():
INFO    plugin-paytest.py: RuntimeError: dictionary changed size during iteration

the timout function is killing the secrete I have a fix will update the PR soonish

@cdecker cdecker force-pushed the paytest-add-forced-timeout branch from 5f7486f to 2a653a1 Compare August 18, 2021 13:10
@cdecker
Copy link
Contributor

cdecker commented Dec 13, 2021

@Mergifyio rebase

especially uncertain about line 240
because I don't know what actually happens on a protocol level if one tries to cancle and htlc that has already been canceled. would be bad if the channel gets failed. it wasn't clearly to me specified at: https://github.com/lightningnetwork/lightning-rfc/blob/master/02-peer-protocol.md#removing-an-htlc-update_fulfill_htlc-update_fail_htlc-and-update_fail_malformed_htlc

if that was a problem anyway we would still have race condition problems if the call comes roughly at the same time when the timout timer triggers anyway
fixed bug that makes the plugin throw an execption after first pending payment is rejected also improved entry in logfile
@cdecker cdecker force-pushed the paytest-add-forced-timeout branch from 2a653a1 to d426ffb Compare December 13, 2021 13:21
@mergify
Copy link

mergify bot commented Dec 13, 2021

rebase

✅ Branch has been successfully rebased

@chrisguida
Copy link
Collaborator

Needs rebase, paytest has been removed from the actively maintained plugins list to get CI running again, but feel free to add it back :)

@chrisguida
Copy link
Collaborator

Closing this as paytest is no longer maintained. Feel free to resurrect it by opening a PR that moves it out of archived :)

@chrisguida chrisguida closed this Jun 6, 2024
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.

3 participants