@remit/backend 0.0.140 → 0.0.142

package/dev-server/keep-alive.test.ts

@@ -0,0 +1,24 @@
+import assert from "node:assert/strict";
+import type { Server } from "node:http";
+import { after, before, it } from "node:test";
+import { UPSTREAM_KEEPALIVE_IDLE_TIMEOUT_SECONDS } from "@remit/apisix/route-table";
+
+let server: Server;
+
+before(async () => {
+	process.env.BETTER_AUTH_SECRET = "a-signing-secret-of-at-least-32-characters";
+	process.env.CORS_ALLOWED_ORIGINS = "https://mail.example.com";
+	process.env.SERVER_PORT = "0";
+	server = (await import("./server.js")).listener;
+});
+
+after(async () => {
+	await new Promise<void>((resolve) => server.close(() => resolve()));
+});
+
+it("keeps an idle socket open longer than the APISIX upstream pool does", () => {
+	assert.ok(
+		server.keepAliveTimeout > UPSTREAM_KEEPALIVE_IDLE_TIMEOUT_SECONDS * 1_000,
+		`backend keepAliveTimeout ${server.keepAliveTimeout}ms must exceed APISIX idle_timeout ${UPSTREAM_KEEPALIVE_IDLE_TIMEOUT_SECONDS}s`,
+	);
+});

package/dev-server/server.ts

@@ -378,4 +378,7 @@ export const listener = app.listen(Number(port), "0.0.0.0", () => {
 	process.send?.("ready");
 });
 
+// Outlasts the idle timeout of every proxy that reuses connections to it.
+listener.keepAliveTimeout = 65_000;
+
 export default app;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@remit/backend",
-  "version": "0.0.140",
+  "version": "0.0.142",
   "description": "Remit Mail Inspector API backend",
   "license": "MIT",
   "author": "",
@@ -54,6 +54,7 @@
     "serve": "node --import tsx dev-server/server.ts"
   },
   "devDependencies": {
+    "@remit/apisix": "*",
     "@types/better-sqlite3": "^7.6.13",
     "@types/express": "^5.0.3",
     "@types/swagger-ui-express": "^4.1.8",

package/src/handlers/sync.ts

@@ -57,9 +57,7 @@ export const triggerSyncSafe = async (
 				// POST /sync asks for a sync of this account by name — the refresh
 				// control, pull-to-refresh, and the client's automatic poll all land
 				// here — so it syncs every mailbox regardless of how recently one
-				// ran. The poll's cadence is floored client-side at the fan-out
-				// gate's window, so this branch cannot be driven faster than the
-				// gate would allow.
+				// ran.
 				explicitRequest: true,
 			});
 			deps.logger.info(