Merge branch 'TBG-FR-fix-createPassportContext-error'

Author: kamilmysliwiec

lib/auth.guard.ts

@@ -21,13 +21,22 @@ export type IAuthGuard = CanActivate & {
   logIn<TRequest extends { logIn: Function } = any>(
     request: TRequest
   ): Promise<void>;
-  handleRequest<TUser = any>(err, user, info, context: ExecutionContext, status?): TUser;
-  getAuthenticateOptions(context: ExecutionContext): IAuthModuleOptions | undefined;
+  handleRequest<TUser = any>(
+    err,
+    user,
+    info,
+    context: ExecutionContext,
+    status?
+  ): TUser;
+  getAuthenticateOptions(
+    context: ExecutionContext
+  ): IAuthModuleOptions | undefined;
 };
 export const AuthGuard: (type?: string | string[]) => Type<IAuthGuard> =
   memoize(createAuthGuard);
 
 const NO_STRATEGY_ERROR = `In order to use "defaultStrategy", please, ensure to import PassportModule in each place where AuthGuard() is being used. Otherwise, passport won't work correctly.`;
+const authLogger = new Logger('AuthGuard');
 
 function createAuthGuard(type?: string | string[]): Type<CanActivate> {
   class MixinAuthGuard<TUser = any> implements CanActivate {
@@ -38,7 +47,7 @@ function createAuthGuard(type?: string | string[]): Type<CanActivate> {
     constructor(@Optional() options?: AuthModuleOptions) {
       this.options = options ?? this.options;
       if (!type && !this.options.defaultStrategy) {
-        new Logger('AuthGuard').error(NO_STRATEGY_ERROR);
+        authLogger.error(NO_STRATEGY_ERROR);
       }
     }
 
@@ -82,6 +91,7 @@ function createAuthGuard(type?: string | string[]): Type<CanActivate> {
 
     handleRequest(err, user, info, context, status): TUser {
       if (err || !user) {
+        authLogger.error(info);
         throw err || new UnauthorizedException();
       }
       return user;

package.json

@@ -20,7 +20,7 @@
     "dist",
     "index.js",
     "index.d.ts"
-  ]
+  ],
   "peerDependencies": {
     "@nestjs/common": "^8.0.0 || ^9.0.0",
     "passport": "^0.4.0 || ^0.5.0 || ^0.6.0"