- Linting (ESLint): Avoid "useless" catch error

- Minor fixes for template files for gitignore/package.json/README
- npm: update devDeps

.eslintrc.js

@@ -5,7 +5,7 @@ module.exports = {
     },
     extends: [
       '@passport-next/eslint-config-passport-next/sauron-node.js',
-      // Override ash-nazg's current preference for ESM
+      // Override eslint-config-passport-next's current preference for ESM
       'plugin:node/recommended-script'
     ],
     settings: {
@@ -74,7 +74,7 @@ module.exports = {
       // Disable until ready to tackle
       'require-jsdoc': 0,
 
-      // Disable current preferences of ash-nazg
+      // Disable current preferences of eslint-config-passport-next
       'import/no-commonjs': 0,
       'node/exports-style': 0,
 

.gitignore

@@ -1,6 +1,5 @@
 reports/
 docs/jsdoc
-.github/docs/jsdoc
 var/
 
 

lib/http/request.js

@@ -32,6 +32,7 @@ const req = exports = module.exports = {};
  *     try {
  *      await req.logIn(user);
  *     } catch (err) {
+ *      console.error(err);
  *      throw err;
  *     }
  *     // session saved

lib/middleware/authenticate.js

@@ -329,6 +329,7 @@ module.exports = function authenticate(passport, name, options, callback) {
         if (options.authInfo !== false) {
           try {
             const tinfo = await passport.transformAuthInfo(info, req);
+            // eslint-disable-next-line require-atomic-updates
             req.authInfo = tinfo;
             complete();
           } catch (err) {

lib/sessionmanager.js

@@ -55,6 +55,7 @@ class SessionManager {
       }
       return;
     }
+    /* eslint-disable require-atomic-updates */
     if (!req._passport.session) {
       req._passport.session = {};
     }
@@ -63,6 +64,7 @@ class SessionManager {
       req.session = {};
     }
     req.session[this._key] = req._passport.session;
+    /* eslint-enable require-atomic-updates */
     // eslint-disable-next-line no-unused-expressions
     cb && cb();
   }

lib/strategies/session.js

@@ -85,6 +85,7 @@ class SessionStrategy extends Strategy {
       } else {
         // TODO: Remove instance access (set by `initialize`)
         const property = req._passport.instance._userProperty || 'user';
+        // eslint-disable-next-line require-atomic-updates
         req[property] = user;
       }
       this.pass();