refactor(chore): fix any types sonar code smells (#208)

fix sonar code smells related to unexpected any types

GH-204

Author: sf-sahil-jassal

src/strategies/types/cognito.types.ts

@@ -1,3 +1,5 @@
+import {AnyObject} from '@loopback/repository';
+
 export namespace Cognito {
   export interface StrategyOptions {
     callbackURL: string;
@@ -16,14 +18,14 @@ export namespace Cognito {
     // eslint-disable-next-line @typescript-eslint/naming-convention
     phone_number?: string;
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    [key: string]: any;
+    [key: string]: any; // NOSONAR
   }
 
   export type VerifyCallback = (
     err?: string | Error,
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    user?: any,
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    info?: any,
+
+    user?: AnyObject,
+
+    info?: AnyObject,
   ) => void;
 }

src/strategies/types/keycloak.types.ts

@@ -1,3 +1,6 @@
+import {AnyObject} from '@loopback/repository';
+import {IAuthUser} from '../../types';
+
 export namespace Keycloak {
   export interface StrategyOptions {
     host: string;
@@ -20,14 +23,14 @@ export namespace Keycloak {
     avatar: string;
     realm: string;
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    [key: string]: any;
+    [key: string]: any; // NOSONAR
   }
 
   export type VerifyCallback = (
     err?: string | Error,
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    user?: any,
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    info?: any,
+
+    user?: IAuthUser,
+
+    info?: AnyObject,
   ) => void;
 }

src/strategies/types/types.ts

@@ -1,21 +1,22 @@
 /* eslint-disable @typescript-eslint/naming-convention */
+import {AnyObject} from '@loopback/repository';
 import {Request} from '@loopback/rest';
-import * as GoogleStrategy from 'passport-google-oauth20';
-import * as AzureADStrategy from 'passport-azure-ad';
-import * as InstagramStrategy from 'passport-instagram';
-import * as FacebookStrategy from 'passport-facebook';
-import * as AppleStrategy from 'passport-apple';
 import * as SamlStrategy from '@node-saml/passport-saml';
+import * as AppleStrategy from 'passport-apple';
 import {DecodedIdToken} from 'passport-apple';
+import * as AzureADStrategy from 'passport-azure-ad';
+import * as FacebookStrategy from 'passport-facebook';
+import * as GoogleStrategy from 'passport-google-oauth20';
+import * as InstagramStrategy from 'passport-instagram';
 import {Cognito, IAuthClient, IAuthSecureClient, IAuthUser} from '../../types';
-import {Keycloak} from './keycloak.types';
 import {Otp} from '../passport';
+import {Keycloak} from './keycloak.types';
 
 export type VerifyCallback = (
   err?: string | Error | null,
   user?: Express.User,
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  info?: any,
+
+  info?: AnyObject,
 ) => void;
 
 export namespace VerifyFunction {

src/strategy-adapter.ts

@@ -2,6 +2,7 @@
 // Node module: @loopback/authentication
 // This file is licensed under the MIT License.
 // License text available at https://opensource.org/licenses/MIT
+import {AnyObject} from '@loopback/repository';
 import {HttpErrors, Request, Response} from '@loopback/rest';
 import {Strategy} from 'passport';
 
@@ -37,8 +38,7 @@ export class StrategyAdapter<T> {
     return new Promise<T | void>((resolve, reject) => {
       // mix-in passport additions like req.logIn and req.logOut
       for (const key in passportRequestMixin) {
-        /* eslint-disable-next-line @typescript-eslint/no-explicit-any */
-        (request as any)[key] = passportRequestMixin[key];
+        (request as AnyObject)[key] = passportRequestMixin[key];
       }
 
       // create a prototype chain of an instance of a passport strategy