fix(strategy): support options as first strategy constructor argument, to support @nestjs/passport pattern

build(package): update package name for @briebug, bump version to 1.3.0

Author: jrista

README.md

@@ -35,6 +35,7 @@ import passportCustom from 'passport-custom';
 const CustomStrategy = passportCustom.Strategy;
 
 passport.use('strategy-name', new CustomStrategy(
+ {},
  function(req, callback) {
  // Do your custom user finding logic here, or set to false based on req object
  callback(null, user);

lib/index.d.ts

@@ -10,7 +10,7 @@ export interface VerifiedCallback {
 }
 
 export declare class Strategy extends PassportStrategy {
- constructor(verify: VerifyCallback);
+ constructor(options: any, verify: VerifyCallback);
  authenticate(req: Request, options?: any): any;
 }
 

lib/strategy.js

@@ -11,7 +11,7 @@ var passport = require('passport-strategy'),
  * Applications must supply a `verify` callback which executes custom authentication logic, and then calls the `done` callback supplying a `user`, which should be set to `false` if the credentials are not valid. If an exception occured, `err` should be set.
  *
  * Examples:
- * passport.use(new CustomStrategy(
+ * passport.use(new CustomStrategy({...},
  * function(req, done) {
  * User.findOne({
  * username: req.body.username
@@ -24,7 +24,7 @@ var passport = require('passport-strategy'),
  * @param {Function} verify Verifies the user.
  * @api public
  */
-function Strategy(verify) {
+function Strategy(options, verify) {
 if (!verify) {
 throw new TypeError('CustomStrategy requires a verify callback');
 }