Prefactor: Rename `StoredWebhook` to `Webhook`
What the developer wrote
Authored by Elias Rohrer
Prefactor: Rename `StoredWebhook` to `Webhook`
What changed, and why it matters
This commit is a simple internal rename of a Rust struct from 'StoredWebhook' to 'Webhook' within the LSPS5 service module. It changes no behavior, logic, or public interfaces and has no security relevance.
No action required. This is a non-functional rename; review can be limited to confirming no accidental logic changes were introduced.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The patch renames a private struct StoredWebhook to Webhook and updates all four references in lightning-liquidity/src/lsps5/service.rs. The struct remains private (struct Webhook), its fields are unchanged, and the surrounding webhook registration/pruning logic is identical. This is a pure refactoring/prefactor commit.
Changed components
lightning-liquidity/src/lsps5/service.rsInspect captured patch +4 / −4
diff --git a/lightning-liquidity/src/lsps5/service.rs b/lightning-liquidity/src/lsps5/service.rs
index 2b86ad3..bb02509 100644
--- a/lightning-liquidity/src/lsps5/service.rs
+++ b/lightning-liquidity/src/lsps5/service.rs
@@ -47,7 +47,7 @@ pub const PRUNE_STALE_WEBHOOKS_INTERVAL_DAYS: Duration = Duration::from_secs(24
/// A stored webhook.
#[derive(Debug, Clone)]
-struct StoredWebhook {
+struct Webhook {
_app_name: LSPS5AppName,
url: LSPS5WebhookUrl,
_counterparty_node_id: PublicKey,
@@ -117,7 +117,7 @@ where
TP::Target: TimeProvider,
{
config: LSPS5ServiceConfig,
- webhooks: Mutex<HashMap<PublicKey, HashMap<LSPS5AppName, StoredWebhook>>>,
+ webhooks: Mutex<HashMap<PublicKey, HashMap<LSPS5AppName, Webhook>>>,
event_queue: Arc<EventQueue>,
pending_messages: Arc<MessageQueue>,
time_provider: TP,
@@ -189,7 +189,7 @@ where
} else {
(now, new_hash_map())
};
- entry.insert(StoredWebhook {
+ entry.insert(Webhook {
_app_name: params.app_name.clone(),
url: params.webhook.clone(),
_counterparty_node_id: counterparty_node_id,
@@ -212,7 +212,7 @@ where
});
}
- entry.insert(StoredWebhook {
+ entry.insert(Webhook {
_app_name: params.app_name.clone(),
url: params.webhook.clone(),
_counterparty_node_id: counterparty_node_id,
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.