P2: RATE_LIMIT_PER_MINUTE env var NaN on invalid input — rate limiting silently disabled #13
Labels
No Label
P0 Critical
P1 High
P2 Medium
P3 Low
accessibility
backend
bug
content
data-integrity
enhancement
frontend
infra
integration
owner
owner-input
performance
performance
phase-7
phase-8
security
seo
ui
ux
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: null/Queue-North-Website#13
Loading…
Reference in New Issue
No description provided.
Delete Branch "%!s(<nil>)"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Severity: P2 — Medium
File: server/index.js ~line 38
Problem: parseInt(process.env.RATE_LIMIT_PER_MINUTE || 5, 10) returns NaN if env var is non-numeric (e.g., abc). Rate limiting silently disabled.
Impact: Rate limiting disabled without warning.
Fix: Add NaN check and fallback to default value with warning log.