Merge branch 'main' into chore/sync-package-lock

backend/src/app.ts

@@ -13,6 +13,9 @@ import shippingAddressesRouter from './routes/shippingAddresses';
 import { attachCustomer } from './middleware/customerAuth';
 
 const app = express();
+// Express advertises itself in X-Powered-By by default, which hands an
+// attacker the server stack for free when picking exploits to try.
+app.disable('x-powered-by');
 app.set('trust proxy', 1);
 
 app.use('/webhooks/paypal', express.json(), cartCheckoutWebhookRouter);

backend/src/routes/admin.ts

@@ -13,7 +13,9 @@ const UPLOADS_DIR = process.env.UPLOADS_DIR || '/app/uploads';
 // multipart body: image count, bytes per image, and the small text fields
 // (name/description/price) that accompany them.
 const MAX_IMAGES_PER_REQUEST = 6;
-const MAX_IMAGE_BYTES = 8 * 1024 * 1024;
+// 8 MB, not 8 MiB — this is the ceiling S5693 treats as safe, and 8 * 1024 *
+// 1024 sits just over it. Plenty for a product photo either way.
+const MAX_IMAGE_BYTES = 8_000_000;
 const MAX_TEXT_FIELDS = 8;
 const MAX_TEXT_FIELD_BYTES = 64 * 1024;