@uppy/companion: remove dependency on express-request-id

[aduh95]

That module is not doing anything special, IMO using it puts more maintenance burden on us than implementing it ourselves.

[socket-security]

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package	New capabilities	Transitives	Size	Publisher
npm/fs-constants@1.0.0	filesystem	0	2.22 kB	mafintosh

🚮 Removed packages: npm/flow-parser@0.236.0, npm/postcss-safe-parser@6.0.0

View full report↗︎

[github-actions]

Diff output files

diff --git a/packages/@uppy/companion/lib/standalone/index.js b/packages/@uppy/companion/lib/standalone/index.js
index db83631..71517a9 100644
--- a/packages/@uppy/companion/lib/standalone/index.js
+++ b/packages/@uppy/companion/lib/standalone/index.js
@@ -2,11 +2,11 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 const express = require("express");
 const qs = require("node:querystring");
+const { randomUUID } = require("node:crypto");
 const helmet = require("helmet");
 const morgan = require("morgan");
 const { URL } = require("node:url");
 const session = require("express-session");
-const addRequestId = require("express-request-id")();
 const RedisStore = require("connect-redis").default;
 const logger = require("../server/logger");
 const redis = require("../server/redis");
@@ -67,7 +67,12 @@ module.exports = function server(inputCompanionOptions) {
     });
     return { query, censored };
   }
-  router.use(addRequestId);
+  router.use((request, response, next) => {
+    const headerName = "X-Request-Id";
+    const oldValue = request.get(headerName);
+    response.set(headerName, oldValue ?? randomUUID());
+    next();
+  });
   // log server requests.
   router.use(morgan("combined"));
   morgan.token("url", (req) => {