Drop unused `LSPS1ServiceEvent::Refund` event
What the developer wrote
Authored by Elias Rohrer
Drop unused `LSPS1ServiceEvent::Refund` event
Turns out this was another variant we didn't actually use anywhere. So
we're dropping it.
What changed, and why it matters
This commit simply removes an unused event variant called Refund from a Lightning Service Provider Specification (LSPS1) event enum. It is a cleanup change with no security relevance: the variant was never used in the code, so removing it cannot introduce or fix a vulnerability.
No action required. This is a benign dead-code removal.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The patch deletes the LSPS1ServiceEvent::Refund variant from lightning-liquidity/src/lsps1/event.rs. The commit message explicitly states the variant was unused. There is no functional code change, no logic modification, and no security-sensitive behavior altered.
Changed components
lightning-liquidity/src/lsps1/event.rsInspect captured patch +0 / −11
diff --git a/lightning-liquidity/src/lsps1/event.rs b/lightning-liquidity/src/lsps1/event.rs
index d78d6d9..1d18842 100644
--- a/lightning-liquidity/src/lsps1/event.rs
+++ b/lightning-liquidity/src/lsps1/event.rs
@@ -174,15 +174,4 @@ pub enum LSPS1ServiceEvent {
/// client.
refund_onchain_address: Option<Address>,
},
- /// If error is encountered, refund the amount if paid by the client.
- ///
- /// **Note: ** This event will *not* be persisted across restarts.
- Refund {
- /// An identifier.
- request_id: LSPSRequestId,
- /// The node id of the client making the information request.
- counterparty_node_id: PublicKey,
- /// The order id of the refunded order.
- order_id: LSPS1OrderId,
- },
}
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.