bhoriuchi / passport-activedirectory

Active Directory strategy for passport.js
29 stars 16 forks source link

fix: fail, not error, on InvalidCredentialsError #16

Closed LoneRifle closed 1 year ago

LoneRifle commented 1 year ago

activedirectory2 will return an InvalidCredentialsError if this strategy is used without integration (ie, config integrated: false), and if invalid credentials are given. Invoke fail() instead of error(), so that the caller knows that this is an auth failure rather than a system issue.

Fixes #5