Changeset e3a5aed in Klonkt for src/routes/admin-stats.js

Timestamp:

06/30/2026 10:11:03 PM (2 months ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

d9c5eb8

Parents:

65a0697

Message:

feat(stats): let the admin exclude their own IP from statistics

Logged-in admins were already skipped, but an admin browsing logged out (incognito, another browser)
still inflated the numbers. Admin -> Statistics now shows your current IP with a one-click "Don't
count my visits" toggle, stored in a stats_exclude_ips app_setting and checked on every pageview
(recordPageview/recordPostView). Trust-proxy gives the real client IP; normalised for matching.

Co-Authored-By: Claude <noreply@…>

File:

• src/routes/admin-stats.js (modified) (3 diffs)

src/routes/admin-stats.js

@@ -14 +14 @@
 import { requireGod } from '../middleware/auth.js';
 import { premiumUnlocked } from '../services/PatreonService.js';
-import { getStats } from '../services/StatsService.js';
+import { getStats, currentIp, getExcludedIps, setExcludedIps } from '../services/StatsService.js';
 
 const router = express.Router();
@@ -32 +32 @@
   }
   const days = [7, 14, 30, 90].includes(parseInt(req.query.days, 10)) ? parseInt(req.query.days, 10) : 14;
+  const myIp = currentIp(req);
   renderPage(req, res, 'pages/admin-stats', {
     pageTitleKey: 'admin.t_stats',
@@ -37 +38 @@
     stats: getStats(days),
     linkClicks,
+    myIp,
+    ipExcluded: !!myIp && getExcludedIps().includes(myIp),
   });
 });
 
+// Toggle whether the admin's current IP is counted in statistics.
+router.post('/exclude-ip', requireGod, (req, res) => {
+  const ip = currentIp(req);
+  if (ip) {
+    const list = getExcludedIps();
+    const i = list.indexOf(ip);
+    if (i >= 0) list.splice(i, 1); else list.push(ip);
+    setExcludedIps(list);
+  }
+  res.redirect('/admin/stats');
+});
+
 export default router;