From 8f57b6ff221791e9c088e225d9eb0e99645ed05d Mon Sep 17 00:00:00 2001
From: elpatron <m.busche@gmail.com>
Date: Tue, 2 Jun 2026 20:54:58 +0200
Subject: [PATCH] Remove diagnostic debug code and backend endpoint

---
 client/src/main.tsx                      |  4 --
 client/src/services/pushNotifications.ts | 52 +++---------------------
 server/src/routes/push.ts                |  5 ---
 3 files changed, 5 insertions(+), 56 deletions(-)

diff --git a/client/src/main.tsx b/client/src/main.tsx
index 152012d..092b49c 100644
--- a/client/src/main.tsx
+++ b/client/src/main.tsx
@@ -14,7 +14,6 @@ import {
   reconcileVersionOnStartup
 } from './services/pwaStartup.ts'
 import { redirectToPasskeyCompatibleHostIfNeeded } from './utils/passkeyHost.ts'
-import { logToBackend } from './services/pushNotifications.ts'
 
 declare global {
   interface Window {
@@ -75,16 +74,13 @@ async function bootstrap(): Promise<void> {
   }
 
   if ('serviceWorker' in navigator && !import.meta.env.DEV) {
-    logToBackend('Attempting manual Service Worker registration...')
     navigator.serviceWorker
       .register('/sw.js', { scope: '/' })
       .then((reg) => {
         console.log('Service Worker registered successfully with scope:', reg.scope)
-        logToBackend('Service Worker registered successfully with scope: ' + reg.scope)
       })
       .catch((err) => {
         console.error('Service Worker registration failed:', err)
-        logToBackend('Service Worker registration failed', err)
       })
   }
 
diff --git a/client/src/services/pushNotifications.ts b/client/src/services/pushNotifications.ts
index 71e2d04..b16fa71 100644
--- a/client/src/services/pushNotifications.ts
+++ b/client/src/services/pushNotifications.ts
@@ -2,29 +2,6 @@ import { apiFetch, apiJson } from './api.js'
 
 const API_BASE = '/api/push'
 
-export async function logToBackend(message: string, error?: any): Promise<void> {
-  try {
-    await fetch(`${API_BASE}/debug-log`, {
-      method: 'POST',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({
-        message,
-        error: error ? {
-          name: error.name,
-          message: error.message,
-          stack: error.stack,
-          ...error
-        } : undefined,
-        userAgent: navigator.userAgent,
-        href: window.location.href,
-        timestamp: new Date().toISOString()
-      })
-    })
-  } catch (err) {
-    console.warn('Failed to send debug log:', err)
-  }
-}
-
 function urlBase64ToUint8Array(base64String: string): Uint8Array {
   const padding = '='.repeat((4 - (base64String.length % 4)) % 4)
   const base64 = (base64String + padding).replace(/-/g, '+').replace(/_/g, '/')
@@ -212,35 +189,23 @@ async function saveSubscriptionToServer(subscription: PushSubscription): Promise
 }
 
 export async function subscribeToPush(): Promise<void> {
-  logToBackend('subscribeToPush called')
   if (!isPushSupported()) {
-    logToBackend('subscribeToPush: push not supported')
     throw new Error('Push notifications are not supported on this device')
   }
 
   // Pre-resolve registration using getRegistrationCompat to prevent ready state hangs
   let registration = cachedRegistration
   if (!registration) {
-    try {
-      logToBackend('subscribeToPush: getting registration...')
-      registration = await getRegistrationCompat()
-      cachedRegistration = registration
-      logToBackend('subscribeToPush: got registration successfully')
-    } catch (err) {
-      logToBackend('subscribeToPush: failed to get registration', err)
-      throw err
-    }
+    registration = await getRegistrationCompat()
+    cachedRegistration = registration
   }
 
   const publicKey = cachedVapidKey || await fetchVapidPublicKey()
   if (!publicKey) {
-    logToBackend('subscribeToPush: no public key available')
     throw new Error('Push notifications are not configured on this server')
   }
 
-  logToBackend('subscribeToPush: requesting permission...')
   const permission = await requestNotificationPermission()
-  logToBackend(`subscribeToPush: permission result: ${permission}`)
   if (permission !== 'granted') {
     throw new Error('Notification permission denied')
   }
@@ -249,7 +214,6 @@ export async function subscribeToPush(): Promise<void> {
   const applicationServerKey = new Uint8Array(keyBytes)
   
   // Always call subscribe with timeout to prevent silent hangs on push network errors
-  logToBackend('subscribeToPush: subscribing via pushManager...')
   const subscribePromise = registration.pushManager.subscribe({
     userVisibleOnly: true,
     applicationServerKey
@@ -257,15 +221,9 @@ export async function subscribeToPush(): Promise<void> {
   const subscribeTimeout = new Promise<never>((_, reject) =>
     setTimeout(() => reject(new Error('Timeout establishing subscription with push service (FCM/APNs)')), 12000)
   )
-  try {
-    const subscription = await Promise.race([subscribePromise, subscribeTimeout])
-    logToBackend('subscribeToPush: subscribed successfully, saving to server...')
-    await saveSubscriptionToServer(subscription)
-    logToBackend('subscribeToPush: saved to server successfully')
-  } catch (err) {
-    logToBackend('subscribeToPush: subscription or save failed', err)
-    throw err
-  }
+  const subscription = await Promise.race([subscribePromise, subscribeTimeout])
+
+  await saveSubscriptionToServer(subscription)
 }
 
 export async function unsubscribeFromPush(): Promise<void> {
diff --git a/server/src/routes/push.ts b/server/src/routes/push.ts
index ec14a07..4c9f648 100644
--- a/server/src/routes/push.ts
+++ b/server/src/routes/push.ts
@@ -22,11 +22,6 @@ router.get('/vapid-public-key', (_req, res) => {
   return res.json({ publicKey })
 })
 
-router.post('/debug-log', (req, res) => {
-  console.log('[CLIENT_DEBUG]', req.body)
-  return res.json({ success: true })
-})
-
 router.use(requireUser)
 
 router.get('/prefs', async (req: any, res) => {