Sign upLogin

feathersjs-ecosystem/feathers-authentication-management

View on GitHub
Star

Showing 18 of 18 total issues

AuthenticationManagementService has 24 functions (exceeds 20 allowed). Consider refactoring.
Open

export class AuthenticationManagementService
  extends AuthenticationManagementBase<AuthenticationManagementData, AllResultTypes, AuthenticationManagementServiceOptions> {
  constructor (
    app: Application,
    options?: Partial<AuthenticationManagementServiceOptions>
Severity: Minor
Found in src/services/AuthenticationManagementService.ts - About 2 hrs to fix

    Function makeClient has 108 lines of code (exceeds 100 allowed). Consider refactoring.
    Open

    function makeClient (app: Application, _options?: Partial<ClientOptions>): AuthenticationManagementClient {
      const options: ClientOptions = Object.assign({}, defaultOptions, _options);
     
      const {
        path
    Severity: Major
    Found in src/client.ts - About 1 hr to fix

      Function resetPassword has 6 arguments (exceeds 5 allowed). Consider refactoring.
      Open

        options: ResetPasswordOptions,
        identifyUser: IdentifyUser,
        tokens: Tokens,
        password: string,
        notifierOptions: NotifierOptions = {},
      Severity: Minor
      Found in src/methods/reset-password.ts - About 35 mins to fix

        Function verifySignupSetPasswordWithShortToken has 6 arguments (exceeds 5 allowed). Consider refactoring.
        Open

          options: VerifySignupSetPasswordWithShortTokenOptions,
          verifyShortToken: string,
          identifyUser: IdentifyUser,
          password: string,
          notifierOptions: NotifierOptions = {},
        Severity: Minor
        Found in src/methods/verify-signup-set-password.ts - About 35 mins to fix

          Function passwordChange has 6 arguments (exceeds 5 allowed). Consider refactoring.
          Open

            options: PasswordChangeOptions,
            identifyUser: IdentifyUser,
            oldPassword: string,
            password: string,
            notifierOptions: NotifierOptions = {},
          Severity: Minor
          Found in src/methods/password-change.ts - About 35 mins to fix

            Function identityChange has 6 arguments (exceeds 5 allowed). Consider refactoring.
            Open

              options: IdentityChangeOptions,
              identifyUser: IdentifyUser,
              password: string,
              changesIdentifyUser: Record<string, unknown>,
              notifierOptions: NotifierOptions = {},
            Severity: Minor
            Found in src/methods/identity-change.ts - About 35 mins to fix

              Function resetPwdWithShortToken has 6 arguments (exceeds 5 allowed). Consider refactoring.
              Open

                options: ResetPwdWithShortTokenOptions,
                resetShortToken: string,
                identifyUser: IdentifyUser,
                password: string,
                notifierOptions: NotifierOptions = {},
              Severity: Minor
              Found in src/methods/reset-password.ts - About 35 mins to fix

                Function eraseVerifyPropsSetPassword has 6 arguments (exceeds 5 allowed). Consider refactoring.
                Open

                    user: User,
                    isVerified: boolean,
                    verifyChanges: VerifyChanges,
                    password: string,
                    skipPasswordHash: boolean,
                Severity: Minor
                Found in src/methods/verify-signup-set-password.ts - About 35 mins to fix

                  Function verifySignupSetPassword has 6 arguments (exceeds 5 allowed). Consider refactoring.
                  Open

                    options: VerifySignupSetPasswordOptions,
                    identifyUser: IdentifyUser,
                    tokens: Tokens,
                    password: string,
                    notifierOptions: NotifierOptions = {},
                  Severity: Minor
                  Found in src/methods/verify-signup-set-password.ts - About 35 mins to fix

                    Avoid too many return statements within this function.
                    Open

                            return await this._sendResetPassword({
                              user: data.value,
                              notifierOptions: data.notifierOptions
                            }, params);
                    Severity: Major
                    Found in src/services/AuthenticationManagementService.ts - About 30 mins to fix

                      Avoid too many return statements within this function.
                      Open

                            return await Promise.reject(err);
                      Severity: Major
                      Found in src/services/AuthenticationManagementService.ts - About 30 mins to fix

                        Avoid too many return statements within this function.
                        Open

                                return await this._identityChange({
                                  changes: data.value.changes,
                                  password: data.value.password,
                                  user: data.value.user,
                                  notifierOptions: data.notifierOptions
                        Severity: Major
                        Found in src/services/AuthenticationManagementService.ts - About 30 mins to fix

                          Avoid too many return statements within this function.
                          Open

                                  return this.options;
                          Severity: Major
                          Found in src/services/AuthenticationManagementService.ts - About 30 mins to fix

                            Avoid too many return statements within this function.
                            Open

                                    return await this._resetPasswordShort({
                                      password: data.value.password,
                                      token: data.value.token,
                                      user: data.value.user,
                                      notifierOptions: data.notifierOptions
                            Severity: Major
                            Found in src/services/AuthenticationManagementService.ts - About 30 mins to fix

                              Avoid too many return statements within this function.
                              Open

                                      return await this._verifySignupSetPasswordShort({
                                        password: data.value.password,
                                        token: data.value.token,
                                        user: data.value.user,
                                        notifierOptions: data.notifierOptions
                              Severity: Major
                              Found in src/services/AuthenticationManagementService.ts - About 30 mins to fix

                                Avoid too many return statements within this function.
                                Open

                                        return await this._resetPasswordLong({
                                          password: data.value.password,
                                          token: data.value.token,
                                          notifierOptions: data.notifierOptions
                                        }, params);
                                Severity: Major
                                Found in src/services/AuthenticationManagementService.ts - About 30 mins to fix

                                  Avoid too many return statements within this function.
                                  Open

                                          return await this._verifySignupSetPasswordLong({
                                            password: data.value.password,
                                            token: data.value.token,
                                            notifierOptions: data.notifierOptions
                                          }, params);
                                  Severity: Major
                                  Found in src/services/AuthenticationManagementService.ts - About 30 mins to fix

                                    Avoid too many return statements within this function.
                                    Open

                                            return await this._passwordChange({
                                              oldPassword: data.value.oldPassword,
                                              password: data.value.password,
                                              user: data.value.user,
                                              notifierOptions: data.notifierOptions
                                    Severity: Major
                                    Found in src/services/AuthenticationManagementService.ts - About 30 mins to fix
                                      Severity
                                      Category
                                      Status
                                      Source
                                      Language