Update to Express v5

Author: pimterry

package.json

@@ -102,7 +102,7 @@
     "@types/common-tags": "1.8.1",
     "@types/connect": "3.4.35",
     "@types/dns2": "1.4.0",
-    "@types/express": "4.17.13",
+    "@types/express": "5.0.6",
     "@types/lodash": "4.14.178",
     "@types/mocha": "8.2.3",
     "@types/native-duplexpair": "^1.0.0",
@@ -176,7 +176,7 @@
     "cors": "^2.8.4",
     "cors-gate": "^1.1.3",
     "destroyable-server": "^1.1.1",
-    "express": "^4.14.0",
+    "express": "^5.2.1",
     "fast-json-patch": "^3.1.1",
     "graphql": "^14.0.2 || ^15.5 || ^16.0.0",
     "graphql-http": "^1.22.0",

src/admin/admin-server.ts

@@ -240,7 +240,7 @@ export class AdminServer<Plugins extends { [key: string]: AdminPlugin<any, any>
         // Dynamically route to mock sessions ourselves, so we can easily add/remove
         // sessions as we see fit later on.
         const sessionRequest = (req: express.Request, res: express.Response, next: express.NextFunction) => {
-            const sessionId = req.params.id;
+            const sessionId = req.params.id as string;
             const sessionRouter = this.sessions[sessionId]?.router;
 
             if (!sessionRouter) {
@@ -297,7 +297,7 @@ export class AdminServer<Plugins extends { [key: string]: AdminPlugin<any, any>
         if (this.server) throw new Error('Admin server already running');
 
         await new Promise<void>((resolve, reject) => {
-            this.server = makeDestroyable(this.app.listen(listenOptions, resolve));
+            this.server = makeDestroyable(this.app.listen(listenOptions, () => resolve()));
 
             this.server.on('error', reject);