fix: stop accepting backup blob credentials from the URL

src/handlers/backup.ts

@@ -1247,19 +1247,14 @@ export async function handleDownloadAdminBackupAttachment(request: Request, env:
   if (!isAdmin(actorUser)) return errorResponse('Forbidden', 403);
 
   try {
-    const url = new URL(request.url);
+    // Read the request body only. Accepting these fields from the query string
-    let input: { blobName?: unknown; masterPasswordHash?: unknown } = {};
+    // would put the master-password authentication hash in the URL, where it is
-    if (request.method === 'POST') {
+    // captured by request logs, browser history and Referer headers.
-      try {
+    let input: { blobName?: unknown; masterPasswordHash?: unknown };
-        input = await request.json<{ blobName?: unknown; masterPasswordHash?: unknown }>();
+    try {
-      } catch {
+      input = await request.json<{ blobName?: unknown; masterPasswordHash?: unknown }>();
-        return errorResponse('Backup attachment download payload is invalid', 400);
+    } catch {
-      }
+      return errorResponse('Backup attachment download payload is invalid', 400);
-    } else {
-      input = {
-        blobName: url.searchParams.get('blobName') || '',
-        masterPasswordHash: url.searchParams.get('masterPasswordHash') || '',
-      };
     }
 
     const verificationError = await requireBackupUserVerification(

src/router-admin-backup.ts

@@ -14,6 +14,7 @@ import {
   handleRunAdminConfiguredBackup,
   handleUpdateAdminBackupSettings,
 } from './handlers/backup';
+import { errorResponse } from './utils/response';
 
 export async function handleAdminBackupRoute(
   request: Request,
@@ -26,8 +27,22 @@ export async function handleAdminBackupRoute(
     return handleAdminExportBackup(request, env, actorUser);
   }
 
-  if (path === '/api/admin/backup/blob' && (method === 'GET' || method === 'POST')) {
+  if (path === '/api/admin/backup/blob') {
-    return handleDownloadAdminBackupAttachment(request, env, actorUser);
+    // POST only: this endpoint requires master-password verification, and a GET
+    // could only carry that credential in the query string, where it would leak
+    // into request logs, proxy logs, browser history and Referer headers.
+    // The credential is the same value clients send to /identity/connect/token,
+    // so a leaked copy is enough to sign in as this admin.
+    if (method === 'POST') {
+      return handleDownloadAdminBackupAttachment(request, env, actorUser);
+    }
+    if (method === 'GET') {
+      return errorResponse(
+        'Use POST with a JSON body for this endpoint. Credentials must not be sent in the URL.',
+        405
+      );
+    }
+    return null;
   }
 
   if (path === '/api/admin/backup/settings') {