upleb.uk

Public git repos — served from a NIP-34 GRASP relay at git.upleb.uk

summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorDanConwayDev <DanConwayDev@protonmail.com>2026-01-14 11:42:05 +0000
committerDanConwayDev <DanConwayDev@protonmail.com>2026-01-14 13:40:03 +0000
commit50000cd9d47681390c3c45feef98fe51c7b79a0f (patch)
tree53ede8cb63ac2c5fe2321a6ecd9c87956537bbc7 /src
parente3792b9abefd43b4594af2640ad4665c006fa3b0 (diff)
Add explicit rate limits and total connection limit
- Make RateLimit explicit in relay builder (500 subs, 60 events/min) - Add NGIT_MAX_CONNECTIONS config option (default: 500) - Update all 4 config locations (src, nix, docs, .env.example) - Fix documentation error: filter limit 5000→500 - Document Phase 2 deferral decision (per-IP enforcement) Addresses primary DoS vector (connection exhaustion) with minimal code. Per-IP rate limiting deferred until abuse detected in production. Related: issue ff38 (git endpoint throttling - separate concern)
Diffstat (limited to 'src')
-rw-r--r--src/config.rs6
-rw-r--r--src/nostr/builder.rs8
2 files changed, 14 insertions, 0 deletions
diff --git a/src/config.rs b/src/config.rs
index 0f0d853..0014003 100644
--- a/src/config.rs
+++ b/src/config.rs
@@ -469,6 +469,11 @@ pub struct Config {
469 /// All events from these authors are blocked from both relay storage and purgatory 469 /// All events from these authors are blocked from both relay storage and purgatory
470 #[arg(long, env = "NGIT_EVENT_BLACKLIST", default_value = "")] 470 #[arg(long, env = "NGIT_EVENT_BLACKLIST", default_value = "")]
471 pub event_blacklist: String, 471 pub event_blacklist: String,
472
473 /// Maximum total connections to the relay (default: 500)
474 /// Prevents connection exhaustion DoS attacks
475 #[arg(long, env = "NGIT_MAX_CONNECTIONS", default_value_t = 500)]
476 pub max_connections: usize,
472} 477}
473 478
474impl Config { 479impl Config {
@@ -703,6 +708,7 @@ impl Config {
703 repository_whitelist: String::new(), 708 repository_whitelist: String::new(),
704 repository_blacklist: String::new(), 709 repository_blacklist: String::new(),
705 event_blacklist: String::new(), 710 event_blacklist: String::new(),
711 max_connections: 500,
706 } 712 }
707 } 713 }
708} 714}
diff --git a/src/nostr/builder.rs b/src/nostr/builder.rs
index c2de1df..ef1b700 100644
--- a/src/nostr/builder.rs
+++ b/src/nostr/builder.rs
@@ -624,6 +624,14 @@ pub async fn create_relay(
624 let relay = LocalRelayBuilder::default() 624 let relay = LocalRelayBuilder::default()
625 .database(database.clone()) 625 .database(database.clone())
626 .write_policy(write_policy.clone()) 626 .write_policy(write_policy.clone())
627 // Explicitly set rate limits (make defaults visible in code)
628 // Per-connection limits: 500 max subscriptions, 60 events/min
629 .rate_limit(RateLimit {
630 max_reqs: 500, // Max concurrent subscriptions per connection
631 notes_per_minute: 60, // Max events per minute per connection
632 })
633 // Total connection limit to prevent DoS attacks
634 .max_connections(config.max_connections)
627 .build(); 635 .build();
628 636
629 tracing::info!( 637 tracing::info!(