Exclude 404 responses from rate-limit counter

[Copilot]

Safari auto-requests /apple-touch-icon.png, /apple-touch-icon-precomposed.png, and /favicon.ico on every page load. These 404s were counted against the 10-failure-per-5-min rate limit, exhausting the budget before any real interaction and triggering 429s on legitimate requests.

Change

Added requestWasSuccessful to the RateLimit config so 404s are treated as non-failures and skipped — only actionable errors (401, 403, 5xx, etc.) consume rate-limit budget.

app.use(RateLimit({
  windowMs: 5 * 60 * 1000,
  max: 10,
  skipSuccessfulRequests: true,
  requestWasSuccessful: (req, res) => res.statusCode < 400 || res.statusCode === 404,
}));

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• scarf.sh
  • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node node ./report.js (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

[cmyers-mieweb]

LGTM