Fix types for authentication method data

AshMWTech · Dec 28, 2023 · f644763 · f644763
1 parent c379471
commit f644763
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/src/types/server.ts b/src/types/server.ts
@@ -3,9 +3,9 @@ import { ExpressErrorResponse, HTTPContext } from './httprouter';
 import { APIInfoObject } from 'documentation';
 import { WebSocketServer } from 'ws';
 
-export type CtxMiddlewareFunction<Context = {}> = (
+export type CtxMiddlewareFunction<Context = {}, Data = any> = (
   ctx: Context & HTTPContext,
-  data?: any,
+  data: Data,
 ) => (express.NextFunction | ExpressErrorResponse | void) | Promise<express.NextFunction | ExpressErrorResponse | void>;
 export type MiddlewareFunction = (
   req: express.Request,

diff --git a/test/authentication/hasPermission.ts b/test/authentication/hasPermission.ts
@@ -5,7 +5,7 @@ interface Data {
   where: string;
 }
 
-export const hasPermission: CtxMiddlewareFunction<Context> = (ctx, data: Data) => {
+export const hasPermission: CtxMiddlewareFunction<Context, Data> = (ctx, data) => {
   console.log(data.where);
   return ctx.next();
 };
diff --git a/test/routes/get.ts b/test/routes/get.ts
@@ -11,7 +11,7 @@ export const configuration: RouteConfig = {
   security: {
     authentication: [
       'loggedIn',
-      authenticationMethod('hasPermission', { penis: false }),
+      authenticationMethod('hasPermission', { where: 'yes' }),
       {
         method: 'cumInAssable',
         data: 'anything',