Merge pull request #1015 from MuckRock/1012-logging

src/app.d.ts

@@ -12,7 +12,9 @@ declare global {
       };
     }
     // interface PageState {}
-    // interface Platform {}
+    interface Platform {
+      context: any;
+    }
   }
 
   namespace svelteHTML {

src/hooks.server.ts

@@ -8,6 +8,7 @@ import * as Sentry from "@sentry/sveltekit";
 import { locale } from "svelte-i18n";
 
 import { DC_BASE } from "./config/config.js";
+import { log } from "$lib/utils/logging";
 
 Sentry.init({
   dsn: env.SENTRY_DSN,
@@ -56,4 +57,17 @@ async function language({ event, resolve }) {
   });
 }
 
-export const handle: Handle = sequence(Sentry.sentryHandle(), language);
+/** @type {import('@sveltejs/kit').Handle} */
+async function logRequest({ event, resolve }) {
+  const response = await resolve(event);
+
+  // logging happens after the response is generated
+  log(event, response);
+  return response;
+}
+
+export const handle: Handle = sequence(
+  Sentry.sentryHandle(),
+  language,
+  logRequest,
+);

src/lib/utils/logging.ts

@@ -0,0 +1,30 @@
+// log requests
+
+import type { RequestEvent } from "@sveltejs/kit";
+
+/**
+ * Log a request inside a Handle function
+ * @param status
+ * @param event
+ */
+export function log(event: RequestEvent, response: Response): void {
+  // be loud about errors
+  const status = response.status;
+  const f = status >= 400 ? console.warn : console.info;
+
+  const { method, url } = event.request;
+  const cache = response.headers.get("cache-control") ?? "";
+  const etag = response.headers.get("etag") ?? "";
+
+  const row = [
+    new Date(),
+    method,
+    url,
+    status,
+    event.route.id,
+    etag,
+    cache,
+  ].filter(Boolean);
+
+  f(...row);
+}