Add option to keep session data.

jaredhanson · May 19, 2022 · a349c2b · a349c2b
1 parent e69834e
commit a349c2b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/lib/http/request.js b/lib/http/request.js
@@ -36,7 +36,7 @@ req.logIn = function(user, options, done) {
     if (typeof done != 'function') { throw new Error('req#login requires a callback function'); }
 
     var self = this;
-    this._sessionManager.logIn(this, user, function(err) {
+    this._sessionManager.logIn(this, user, options, function(err) {
       if (err) { self[property] = null; return done(err); }
       done();
     });

diff --git a/lib/sessionmanager.js b/lib/sessionmanager.js
@@ -21,6 +21,7 @@ SessionManager.prototype.logIn = function(req, user, options, cb) {
   if (!req.session) { return cb(new Error('Login sessions require session support. Did you forget to use `express-session` middleware?')); }
 
   var self = this;
+  var prevSession = req.session;
 
   // regenerate the session, which is good practice to help
   // guard against forms of session fixation
@@ -33,6 +34,9 @@ SessionManager.prototype.logIn = function(req, user, options, cb) {
       if (err) {
         return cb(err);
       }
+      if (options.keepSessionData) {
+        Object.assign(req.session, prevSession);
+      }
       if (!req.session[self._key]) {
         req.session[self._key] = {};
       }