work

Author: dmail

dist/js/client_reporter.js

@@ -178,11 +178,19 @@ const setup = () => {
   // only carries the id, tab, activities and log entries.
   let pendingLogs = [];
   let pendingActivities = [];
+  // The server answering that the endpoint does not exist means there is no
+  // monitoring behind this origin anymore (the dev server was restarted
+  // without it, a test server took the port): reporting stops for the rest of
+  // this page's life rather than knocking at every heartbeat.
+  let endpointGone = false;
   const post = async ({ beacon = false, closing = false } = {}) => {
     const logs = pendingLogs;
     const activities = pendingActivities;
     pendingLogs = [];
     pendingActivities = [];
+    if (endpointGone) {
+      return;
+    }
     const payload = JSON.stringify({
       clientId,
       // The server reads browser/OS from the request headers, but a headless
@@ -201,12 +209,15 @@ const setup = () => {
       return;
     }
     try {
-      await nativeFetch(REPORT_ENDPOINT, {
+      const response = await nativeFetch(REPORT_ENDPOINT, {
         method: "POST",
         headers: { "content-type": "application/json" },
         body: payload,
         keepalive: true,
       });
+      if (response.status === 404 || response.status === 405) {
+        endpointGone = true;
+      }
     } catch {
       // dev server gone or offline — dropping the report is acceptable.
     }

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "41.4.19",
+  "version": "41.4.20",
   "type": "module",
   "description": "Tool to develop, test and build js projects",
   "repository": {
@@ -78,7 +78,7 @@
     "@jsenv/plugin-minification": "1.7.10",
     "@jsenv/plugin-supervisor": "1.8.13",
     "@jsenv/plugin-transpilation": "1.6.1",
-    "@jsenv/server": "17.6.2",
+    "@jsenv/server": "17.6.3",
     "@jsenv/sourcemap": "1.4.2",
     "react-table": "7.8.0"
   },

packages/backend/database-manager/package.json

@@ -30,7 +30,7 @@
   },
   "dependencies": {
     "@jsenv/database": "0.0.15",
-    "@jsenv/server": "17.6.2",
+    "@jsenv/server": "17.6.3",
     "@jsenv/urls": "2.9.10",
     "@tanstack/table-core": "8.21.3",
     "jsonwebtoken": "9.0.3",

packages/backend/server/dist/jsenv_server.js

@@ -6452,7 +6452,10 @@ It should be should be one of route.${routePropertyName}: ${availableValues.join
           }
         }
         if (!route.matchMethod(request.method)) {
-          if (!route.isFallback) {
+          // a 405 asserts the resource exists with other methods: a route
+          // matching any resource ("GET *") cannot assert that, so it does
+          // not turn an unknown resource into a 405
+          if (!route.isFallback && route.resource !== "*") {
             wouldHaveMatched.methodSet.add(route.method);
           }
           continue;

packages/backend/server/docs/handling_requests.md

@@ -32,7 +32,7 @@ Routes are tried in order. `fetch` can be async; returning `null` or `undefined`
 
 When no route answers, the router builds the response from what almost matched:
 
-- **405 Method Not Allowed** (with an `allow` header) when the resource matched routes for other methods,
+- **405 Method Not Allowed** (with an `allow` header) when the resource matched routes for other methods — a catch-all `*` route does not count, it says nothing about the resource,
 - **415 Unsupported Media Type** when a POST/PATCH/PUT route wanted another `content-type` (see `acceptedMediaTypes` below),
 - **406 Not Acceptable** when the route cannot produce what the request accepts (see [content negotiation](./content_negotiation.md)),
 - **426 Upgrade Required** for a websocket route requested without an upgrade,

packages/backend/server/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/server",
-  "version": "17.6.2",
+  "version": "17.6.3",
   "type": "module",
   "description": "A modern Node.js HTTP server with declarative routing, content negotiation, and WebSocket support",
   "repository": {

packages/backend/server/src/router/router.js

@@ -355,7 +355,10 @@ It should be should be one of route.${routePropertyName}: ${availableValues.join
           }
         }
         if (!route.matchMethod(request.method)) {
-          if (!route.isFallback) {
+          // a 405 asserts the resource exists with other methods: a route
+          // matching any resource ("GET *") cannot assert that, so it does
+          // not turn an unknown resource into a 405
+          if (!route.isFallback && route.resource !== "*") {
             wouldHaveMatched.methodSet.add(route.method);
           }
           continue;

packages/related/cli/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/cli",
-  "version": "0.3.146",
+  "version": "0.3.147",
   "type": "module",
   "description": "Command Line Interface for jsenv",
   "repository": {

packages/related/cli/template-node-package/package.json

@@ -13,7 +13,7 @@
   },
   "devDependencies": {
     "@jsenv/assert": "4.5.7",
-    "@jsenv/core": "41.4.19",
+    "@jsenv/core": "41.4.20",
     "@jsenv/eslint-config-relax": "1.9.0",
     "@jsenv/test": "3.7.34",
     "eslint": "9.39.2",

packages/related/cli/template-web-components/package.json

@@ -18,7 +18,7 @@
     "@babel/plugin-syntax-import-attributes": "7.29.7",
     "@jsenv/custom-elements-redefine": "0.1.0",
     "@jsenv/assert": "4.5.7",
-    "@jsenv/core": "41.4.19",
+    "@jsenv/core": "41.4.20",
     "@jsenv/plugin-bundling": "2.10.18",
     "@jsenv/plugin-minification": "1.7.10",
     "@jsenv/eslint-config-relax": "1.9.0",