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

Ensure HTLC signature transport demands conform to BOLT wire-message size limitations #1149

Closed
2 tasks
dstadulis opened this issue Oct 11, 2024 · 1 comment · Fixed by #1132
Closed
2 tasks
Assignees

Comments

@dstadulis
Copy link
Collaborator

dstadulis commented Oct 11, 2024

Solution Space

The daemon abstract settlement into one logical message (whose size might be in excess of the BOLT limitation) but the logical message would be fragmented into multiple over-the-wire chunks which conform to the BOLT maximum?
This design would mimic TCP fragmenting

Solution

  • MVP Solution: Place limitations on in-flight HTLCs
  • Long term solution: Fragment messages akin to TCP fragmenting

Limiting UTXOs in Funding Output:

Reason: Ensure conformance with message-size limits on protocol level
Cause:
- Each HTLC may reference multiple asset UTXOs from funding output
- Each referenced input needs a signature, increasing transmission data
Alternative Solutions:
- Implement SIGHASH_NOINPUT on protocol layer
- Allows sending second-level signatures alongside HTLCs
- Requires stable pkScript for next commitment (achieved with per-commitment-secret)
- Limit total number of UTXOs within funding output
- Asset ID is not the limiting factor, but rather total UTXO count

[Original commentary]

@dstadulis
Copy link
Collaborator Author

due to wire message limit

@dstadulis dstadulis moved this from 🆕 New to 🏗 In progress in Taproot-Assets Project Board Oct 29, 2024
@dstadulis dstadulis changed the title Ensure transaction transmissibility due to HTLC signature-creation requirements by limiting transaction's asset UTXO input count Ensure HTLC signature transport demands conform to BOLT wire-message size limitations Oct 29, 2024
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Taproot-Assets Project Board Nov 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

2 participants