AI-generated analysisPublished automatically and not human-verified. Validated context appears in community notes below.
← Watch feed
Informational 15 Bitcoin

tests: Add tests for extra_tlvs in hook

Public commit record

What the developer wrote

Authored by Peter Neuroth

78/100 · Adequate
tests: Add tests for extra_tlvs in hook

Adds some testcases for custom tlvs, set by a htlc_accepted_hook. We
check that the custom tlvs replace the update_add_htlc_tlvs and get
forwarded to the peer. We also check that a malformed tlv will result in
a **BROKEN** behaviour.

Signed-off-by: Peter Neuroth <pet.v.ne@gmail.com>
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification
The short version

What changed, and why it matters

This commit only adds new automated tests and a test plugin. It does not change any production code, so it cannot by itself introduce a security vulnerability or fix one. The tests exercise an existing feature that lets plugins attach custom data fields to forwarded Lightning payments, and verify that malformed data triggers a 'BROKEN' log warning.

Recommended action

No security action required; treat as routine test coverage. Review the related production feature separately if concerned about plugin-controlled TLV injection.

Security signals we found

No strong security signals were identified.

Risk score

Why this scored 15/100

Our methodology →
Potential impact 0/30
Exploitability 0/25
Stealth signal 0/15
Affected reach 0/15
Confidence 10/10
Evidence quality 5/5
Human-validated context

Community notes

Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.

No validated notes yet.

The AI analysis stands alone for now. Submit a note if you can add evidence or important context.