add handling for unset request types in endpoint

doc/authentication.md

@@ -237,6 +237,7 @@ pub fn main() !void {
         .path = "/test",
         .get = endpoint_http_get,
         .unauthorized = endpoint_http_unauthorized,
+        .unset = Endpoint.dummy_handler,
     });
 
     // create authenticator

examples/endpoint/stopendpoint.zig

@@ -14,6 +14,7 @@ pub fn init(
         .ep = zap.Endpoint.init(.{
             .path = path,
             .get = get,
+            .unset = zap.Endpoint.dummy_handler,
         }),
     };
 }

examples/endpoint_auth/endpoint_auth.zig

@@ -42,6 +42,7 @@ pub fn main() !void {
         .path = "/test",
         .get = endpoint_http_get,
         .unauthorized = endpoint_http_unauthorized,
+        .unset = zap.Endpoint.dummy_handler,
     });
 
     // create authenticator

examples/middleware_with_endpoint/middleware_with_endpoint.zig

@@ -144,6 +144,7 @@ const HtmlEndpoint = struct {
             .ep = zap.Endpoint.init(.{
                 .path = "/doesn't+matter",
                 .get = get,
+                .unset = zap.Endpoint.dummy_handler,
             }),
         };
     }

src/endpoint.zig

@@ -30,6 +30,8 @@ pub const Settings = struct {
     options: ?RequestFn = null,
     /// Only applicable to Authenticating Endpoint: handler for unauthorized requests
     unauthorized: ?RequestFn = null,
+    // callback to any unset request type
+    unset: ?RequestFn = null,
 };
 
 settings: Settings,
@@ -41,19 +43,20 @@ pub fn init(s: Settings) Endpoint {
     return .{
         .settings = .{
             .path = s.path,
-            .get = s.get orelse &nop,
-            .post = s.post orelse &nop,
-            .put = s.put orelse &nop,
-            .delete = s.delete orelse &nop,
-            .patch = s.patch orelse &nop,
-            .options = s.options orelse &nop,
-            .unauthorized = s.unauthorized orelse &nop,
+            .get = s.get orelse s.unset orelse @panic("Endpoint handler `.get` is unset, and no `.unset` handler is provided."),
+            .post = s.post orelse s.unset orelse @panic("Endpoint handler `.post` is unset, and no `.unset` handler is provided."),
+            .put = s.put orelse s.unset orelse @panic("Endpoint handler `.put` is unset, and no `.unset` handler is provided."),
+            .delete = s.delete orelse s.unset orelse @panic("Endpoint handler `.delete` is unset, and no `.unset` handler is provided."),
+            .patch = s.patch orelse s.unset orelse @panic("Endpoint handler `.patch` is unset, and no `.unset` handler is provided."),
+            .options = s.options orelse s.unset orelse @panic("Endpoint handler `.options` is unset, and no `.unset` handler is provided."),
+            .unauthorized = s.unauthorized orelse s.unset orelse @panic("Endpoint handler `.unauthorized` is unset, and no `.unset` handler is provided."),
+            .unset = s.unset,
         },
     };
 }
 
 // no operation. Dummy handler function for ignoring unset request types.
-fn nop(self: *Endpoint, r: Request) void {
+pub fn dummy_handler(self: *Endpoint, r: Request) void {
     _ = self;
     _ = r;
 }
@@ -98,6 +101,7 @@ pub fn Authenticating(comptime Authenticator: type) type {
                     .patch = if (e.settings.patch != null) patch else null,
                     .options = if (e.settings.options != null) options else null,
                     .unauthorized = e.settings.unauthorized,
+                    .unset = e.settings.unset,
                 }),
             };
         }
@@ -224,7 +228,7 @@ pub fn Authenticating(comptime Authenticator: type) type {
                         return;
                     }
                 },
-                .AuthOK => authEp.ep.settings.put.?(authEp.ep, r),
+                .AuthOK => authEp.ep.settings.options.?(authEp.ep, r),
                 .Handled => {},
             }
         }

src/tests/test_auth.zig

@@ -191,6 +191,7 @@ test "BearerAuthSingle authenticateRequest OK" {
         .path = "/test",
         .get = endpoint_http_get,
         .unauthorized = endpoint_http_unauthorized,
+        .unset = Endpoint.dummy_handler,
     });
 
     // create authenticator
@@ -244,6 +245,7 @@ test "BearerAuthSingle authenticateRequest test-unauthorized" {
         .path = "/test",
         .get = endpoint_http_get,
         .unauthorized = endpoint_http_unauthorized,
+        .unset = Endpoint.dummy_handler,
     });
 
     const Set = std.StringHashMap(void);
@@ -301,6 +303,7 @@ test "BearerAuthMulti authenticateRequest OK" {
         .path = "/test",
         .get = endpoint_http_get,
         .unauthorized = endpoint_http_unauthorized,
+        .unset = Endpoint.dummy_handler,
     });
 
     // create authenticator
@@ -352,6 +355,7 @@ test "BearerAuthMulti authenticateRequest test-unauthorized" {
         .path = "/test",
         .get = endpoint_http_get,
         .unauthorized = endpoint_http_unauthorized,
+        .unset = Endpoint.dummy_handler,
     });
 
     // create authenticator
@@ -403,6 +407,7 @@ test "BasicAuth Token68 authenticateRequest" {
         .path = "/test",
         .get = endpoint_http_get,
         .unauthorized = endpoint_http_unauthorized,
+        .unset = Endpoint.dummy_handler,
     });
     // create a set of Token68 entries
     const Set = std.StringHashMap(void);
@@ -459,6 +464,7 @@ test "BasicAuth Token68 authenticateRequest test-unauthorized" {
         .path = "/test",
         .get = endpoint_http_get,
         .unauthorized = endpoint_http_unauthorized,
+        .unset = Endpoint.dummy_handler,
     });
     // create a set of Token68 entries
     const Set = std.StringHashMap(void);
@@ -514,6 +520,7 @@ test "BasicAuth UserPass authenticateRequest" {
         .path = "/test",
         .get = endpoint_http_get,
         .unauthorized = endpoint_http_unauthorized,
+        .unset = Endpoint.dummy_handler,
     });
 
     // create a set of User -> Pass entries
@@ -580,6 +587,7 @@ test "BasicAuth UserPass authenticateRequest test-unauthorized" {
         .path = "/test",
         .get = endpoint_http_get,
         .unauthorized = endpoint_http_unauthorized,
+        .unset = Endpoint.dummy_handler,
     });
 
     // create a set of User -> Pass entries

src/zap.zig

@@ -33,6 +33,7 @@ pub const Tls = @import("tls.zig");
 ///             .ep = zap.Endpoint.init(.{
 ///                 .path = path,
 ///                 .get = get,
+///                 .unset = Endpoint.dummy_handler,
 ///             }),
 ///         };
 ///     }