From 976a2519a76756d463ddd6f9c5de71f81cc9fa4f Mon Sep 17 00:00:00 2001
From: Hailey <me@haileyok.com>
Date: Sun, 31 Aug 2025 14:59:11 -0700
Subject: [PATCH] improve logging

---
 bskylink/src/cache/safelinkClient.ts |  5 +----
 bskylink/src/routes/redirect.ts      | 10 ++++------
 2 files changed, 5 insertions(+), 10 deletions(-)

diff --git a/bskylink/src/cache/safelinkClient.ts b/bskylink/src/cache/safelinkClient.ts
index 94deb492ea..edd3890259 100644
--- a/bskylink/src/cache/safelinkClient.ts
+++ b/bskylink/src/cache/safelinkClient.ts
@@ -52,13 +52,10 @@ export class SafelinkClient {
         {error: e, inputUrl: link},
         'failed to normalize looked up link',
       )
-
+      // fail open
       return 'ok'
     }
 
-    redirectLogger.info(url)
-    redirectLogger.info(domain)
-
     const urlRule = this.urlCache.get(url)
     if (urlRule) {
       return urlRule
diff --git a/bskylink/src/routes/redirect.ts b/bskylink/src/routes/redirect.ts
index 1ed2b0755a..681dc0bb90 100644
--- a/bskylink/src/routes/redirect.ts
+++ b/bskylink/src/routes/redirect.ts
@@ -54,7 +54,7 @@ export default function (ctx: AppContext, app: Express) {
         if (rule !== 'ok') {
           switch (rule.action) {
             case 'whitelist':
-              redirectLogger.info(`Whitelist rule matched for ${rule.url}`)
+              redirectLogger.info({rule}, 'Whitelist rule matched')
               break
             case 'block':
               html = linkWarningLayout(
@@ -65,7 +65,7 @@ export default function (ctx: AppContext, app: Express) {
                 }),
               )
               res.setHeader('Cache-Control', 'no-store')
-              redirectLogger.info(`Block rule matched for ${rule.url}`)
+              redirectLogger.info({rule}, 'Block rule matched')
               break
             case 'warn':
               html = linkWarningLayout(
@@ -76,12 +76,10 @@ export default function (ctx: AppContext, app: Express) {
                 }),
               )
               res.setHeader('Cache-Control', 'no-store')
-              redirectLogger.info(`Warn rule matched for ${rule.url}`)
+              redirectLogger.info({rule}, 'Warn rule matched')
               break
             default:
-              redirectLogger.warn(
-                `${rule.action} rule (an unknown rule) matched for ${rule.url}`,
-              )
+              redirectLogger.warn({rule}, 'Unknown rule matched')
           }
         }
       }