fix: change skipS3Storage from || to && for omit headers

Only skip S3 storage when BOTH request AND response are omitted.
Previously, omitting just one would skip S3 entirely, preventing
the other from being visible in Jawn.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: 2 people

worker/src/lib/dbLogger/DBLoggable.ts

@@ -836,10 +836,10 @@ export class DBLoggable {
       // Ignore parsing errors - usage will be extracted later by Jawn if body is stored
     }
 
-    // Skip S3 storage if omit headers are set
-    // Note: Free tier limit exceeded + PTB requests will still log but skip S3
+    // Skip S3 storage only if BOTH request and response are omitted
+    // If only one is omitted, still store to S3 - Jawn will respect the omit flags
     const skipS3Storage =
-      requestHeaders?.omitHeaders?.omitRequest === true ||
+      requestHeaders?.omitHeaders?.omitRequest === true &&
       requestHeaders?.omitHeaders?.omitResponse === true;
 
     if (S3_ENABLED === "true" && !skipS3Storage) {