Updated npm deps and updated from passport-next/skel

passport-next · Sep 10, 2019 · 1419528 · 1419528
1 parent bc04113
commit 1419528
Show file tree

Hide file tree

Showing 10 changed files with 6,286 additions and 6,090 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -5,7 +5,7 @@ node_js:
   - "12"
 
 script:
-  - "npm run coverage"
+  - "npm run test"
   - "npm run lint"
 
 after_success:

diff --git a/lib/authenticator.js b/lib/authenticator.js
@@ -103,7 +103,7 @@ class Authenticator {
    *
    *     passport.framework(require('hapi-passport')());
    *
-   * @param {object} fw
+   * @param {PlainObject} fw
    * @returns {Authenticator} for chaining
    * @public
    */
@@ -167,7 +167,7 @@ class Authenticator {
    *     });
    *
    * @param {string} strategy
-   * @param {object} options
+   * @param {PlainObject} options
    * @param {GenericCallback} callback
    * @returns {GenericCallback} middleware
    * @public
@@ -192,7 +192,7 @@ class Authenticator {
    *    passport.authorize('twitter-authz', { failureRedirect: '/account' });
    *
    * @param {string} strategy
-   * @param {object} options
+   * @param {PlainObject} options
    * @param {GenericCallback} callback
    * @returns {GenericCallback} middleware
    * @public
@@ -230,7 +230,7 @@ class Authenticator {
    *     app.use(passport.initialize());
    *     app.use(passport.session());
    *
-   * @param {object} options
+   * @param {PlainObject} options
    * @returns {GenericCallback} middleware
    * @public
    */
@@ -258,7 +258,7 @@ class Authenticator {
    *
    * @example
    *
-   *     passport.serializeUser(function(user, done) {
+   *     passport.serializeUser(function (user, done) {
    *       done(null, user.id);
    *     });
    *
@@ -320,7 +320,7 @@ class Authenticator {
    *
    * @example
    *
-   *     passport.deserializeUser(function(id, done) {
+   *     passport.deserializeUser(function (id, done) {
    *       User.findById(id, function (err, user) {
    *         done(err, user);
    *       });
@@ -411,7 +411,7 @@ class Authenticator {
    *
    * @example
    *
-   *     passport.transformAuthInfo(function(info, done) {
+   *     passport.transformAuthInfo(function (info, done) {
    *       Client.findById(info.clientID, function (err, client) {
    *         info.client = client;
    *         done(err, info);

diff --git a/lib/framework/connect.js b/lib/framework/connect.js
@@ -13,7 +13,7 @@ const authenticate = require('../middleware/authenticate');
  * This module provides support for using Passport with Express.  It exposes
  * middleware that conform to the `fn(req, res, next)` signature.
  *
- * @return {Object}
+ * @returns {Object}
  * @protected
  */
 

diff --git a/lib/middleware/authenticate.js b/lib/middleware/authenticate.js
@@ -64,8 +64,8 @@ const AuthenticationError = require('../errors/authenticationerror');
  *     passport.authenticate('twitter');
  *
  * @param {Authenticator} passport
- * @param {string|Array} name
- * @param {object} options
+ * @param {string|GenericArray} name
+ * @param {PlainObject} options
  * @param {GenericCallback} callback
  * @returns {GenericCallback}
  * @public

diff --git a/lib/middleware/initialize.js b/lib/middleware/initialize.js
@@ -28,17 +28,17 @@
  *     app.use(passport.initialize());
  *     app.use(passport.session());
  *
- *     passport.serializeUser(function(user, done) {
+ *     passport.serializeUser(function (user, done) {
  *       done(null, user.id);
  *     });
  *
- *     passport.deserializeUser(function(id, done) {
+ *     passport.deserializeUser(function (id, done) {
  *       User.findById(id, function (err, user) {
  *         done(err, user);
  *       });
  *     });
  *
- * @return {Function}
+ * @returns {GenericCallback}
  * @public
  */