send Allow header on 405 Method Not Allowed responses

diff --git a/CHANGELOG.md b/CHANGELOG.md
index cf32abe9fbe1485b2869631e73cd91f1a20d8c49..a3b048b9d256f31e56cf7a3ba950d36afdfff283 100644 (file)
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -5,6 +5,7 @@ Releases and notable changes to this project are documented here.
 ## [Unreleased]
 
 - support naming route paths, and rendering those named paths with parameter substitution
+- send Allow header on 405 Method Not Allowed responses
 
 ## [v2.1.3] - 2025-03-28
 

diff --git a/lib/dingus.js b/lib/dingus.js
index 2c521db908db55200be4440e4061d995617830d0..31c3a101e1a7e30701fdba2ba1db1296fcce77d7 100644 (file)
--- a/lib/dingus.js
+++ b/lib/dingus.js
@@ -359,6 +359,7 @@ class Dingus {
           ({ handler, handlerArgs } = this._determineHeadHandler(req, res, ctx, pathPart));
         } else {
           handler = this.handlerMethodNotAllowed.bind(this);
+          res.setHeader(Enum.Header.Allow, e.methods.join(', '));
         }
       } else {
         this.logger.error(_scope, 'unexpected error', { error: e });
@@ -386,6 +387,7 @@ class Dingus {
     } catch (e) {
       if (e instanceof RouterNoMethodError) {
         handler = this.handlerMethodNotAllowed.bind(this);
+        res.setHeader(Enum.Header.Allow, e.methods.join(', '));
       } else {
         this.logger.error(_scope, 'unexpected error', { error: e });
         handler = this.handlerInternalServerError.bind(this);
@@ -808,7 +810,7 @@ class Dingus {
    * @param {string} newPath url to redirect to
    * @param {number=} statusCode status code to use for redirect, default 307
    */
-  async handlerRedirect(req, res, ctx, newPath, statusCode = 307) {
+  async handlerRedirect(req, res, ctx, newPath, statusCode = Enum.HTTPStatusCode.TemporaryRedirect) {
     this.setResponseType(this.responseTypes, req, res, ctx);
     res.setHeader(Enum.Header.Location, newPath);
     res.statusCode = statusCode;

diff --git a/lib/enum.js b/lib/enum.js
index 5f63ea6d662ac0473ae21e58dbb24ab7f117cc9b..77d2c02fd5179be1b6c38824c3b3483e67ae87d5 100644 (file)
--- a/lib/enum.js
+++ b/lib/enum.js
@@ -202,6 +202,7 @@ const ErrorResponseProxy = new Proxy(ErrorResponse, {
 const Header = {
   Accept: 'Accept',
   AcceptEncoding: 'Accept-Encoding',
+  Allow: 'Allow',
   CacheControl: 'Cache-Control',
   ContentEncoding: 'Content-Encoding',
   ContentLength: 'Content-Length',

diff --git a/lib/errors.js b/lib/errors.js
index 942ecc0f684656059f4e0fad66288db7c57f2647..0aec63310cda4737bb430223c49743ee7f52ff05 100644 (file)
--- a/lib/errors.js
+++ b/lib/errors.js
@@ -33,6 +33,10 @@ class RouterNoPathError extends RouterError {
 }
 
 class RouterNoMethodError extends RouterError {
+  constructor(methods, ...args) {
+    super(...args);
+    this.methods = methods;
+  }
 }
 
 module.exports = {

diff --git a/lib/router/index.js b/lib/router/index.js
index af17a4dfe3b82d21f18c3256d59aaf511fc3e07d..0e2a4e374e339ad2c6790a50f93badc4d4179aff 100644 (file)
--- a/lib/router/index.js
+++ b/lib/router/index.js
@@ -290,7 +290,7 @@ class Router {
       if ('*' in matchedPath[kPathMethods]) {
         return matchedPath[kPathMethods]['*'];
       }
-      throw new RouterNoMethodError();
+      throw new RouterNoMethodError(Object.keys(matchedPath[kPathMethods]));
     }
     ctx.unmatchedPath = pathParts;
     throw new RouterNoPathError();

diff --git a/test/lib/dingus.js b/test/lib/dingus.js
index 8023436c6a25086b11555157af554a0ab4c024fe..e68878ae3383d5e041604b60748abb97b5138f96 100644 (file)
--- a/test/lib/dingus.js
+++ b/test/lib/dingus.js
@@ -513,6 +513,7 @@ describe('Dingus', function () {
       assert(!stubHandler.called);
       assert(dingus.handlerMethodNotAllowed.called);
       assert(!dingus.handlerNotFound.called);
+      assert(res.setHeader.called);
     });
     it('does not lookup nonexistent path', async function () {
       req.url = '/foo/bar';