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
[1.1] Add funding_cancelled
message.
#349
Conversation
4510221
to
d0d11eb
Compare
@@ -306,4 +306,5 @@ offerer | |||
offerer's | |||
incentivize | |||
redemptions | |||
vbytes | |||
cancelled | |||
vbytes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: \n
needed :-)
@@ -19,6 +19,7 @@ These flags may only be used in the `init` message: | |||
| 0/1 | `option-data-loss-protect` | Requires or supports extra `channel_reestablish` fields | [BOLT #2](02-peer-protocol.md#message-retransmission) | | |||
| 3 | `initial_routing_sync` | Indicates that the sending node needs a complete routing information dump | [BOLT #7](07-routing-gossip.md#initial-sync) | | |||
| 4/5 | `option_upfront_shutdown_script` | Commits to a shutdown scriptpubkey when opening | [BOLT #2](02-peer-protocol.md#the-open_channel-message) | | |||
| 8/9 | `option-funding-cancelled` | Requires or supports `funding_cancelled` message | [BOLT #2](02-peer-protocol.md#the-funding_cancelled-message) | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: Doesn't "require" imply "support" in this context? Would Supports funding_cancelled message
be enough?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Even bit requires the feature, odd bit supports if possible ("its ok to be odd").
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See also bits 0/1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ZmnSCPxj Ah, my bad! Missed the 8/9
. Sorry! :-)
Closing: |
This proposed new message indicates that the funding transaction can never confirm. For example, to implement replace-by-fee funding transactions, or to implement multi-channel funding transactions (see rationale of message for details).
This is inteded for BOLT 1.1