Improve error message when session support isn't available.

BaileyFirman · Dec 15, 2021 · c2091b6 · c2091b6
1 parent 444b71a
commit c2091b6
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/strategies/session.js b/lib/strategies/session.js
@@ -43,7 +43,7 @@ util.inherits(SessionStrategy, Strategy);
  * @api protected
  */
 SessionStrategy.prototype.authenticate = function(req, options) {
-  if (!req._passport) { return this.error(new Error('passport.initialize() middleware not in use')); }
+  if (!req.session) { return this.error(new Error('Login sessions require session support. Did you forget to use `express-session` middleware?')); }
   options = options || {};
 
   var self = this, 

diff --git a/test/strategies/session.test.js b/test/strategies/session.test.js
@@ -257,7 +257,7 @@ describe('SessionStrategy', function() {
 
     it('should error', function() {
       expect(error).to.be.an.instanceOf(Error);
-      expect(error.message).to.equal('passport.initialize() middleware not in use');
+      expect(error.message).to.equal('Login sessions require session support. Did you forget to use `express-session` middleware?');
     });
 
     it('should not set user on request', function() {