login-plus 1.6.3 → 1.7.1

package/lib/login-plus.js

@@ -216,10 +216,21 @@ loginPlus.Manager.prototype.init = function init(app,opts){
         }
     });
     app.get(opts.baseUrl+opts.logoutUrlPath, function(req,res,next){
-        req.logout();
-        req.session.returnTo = null;
-        res.redirect(opts.redirBaseUrl+opts.logoutRedirect);
+        req.logout(function(err) {
+            if (err) { return next(err); }
+            req.session.returnTo = null;
+            res.redirect(opts.redirBaseUrl+opts.logoutRedirect);
+        });
     });
+    /*
+    app.post(opts.baseUrl+opts.loginUrlPath, function(req,res,next){
+        if(req.query["return-to"] && req.session){
+            console.log('XXXXXX- CHECK LOGIN PLUS', req.query["return-to"]);
+            req.session.returnTo = req.query["return-to"];
+        }
+        next();
+    });
+    */
     if(!opts.skipCheckAlreadyLoggedIn){
         checkAlreadyLoggedIn(app, opts);
     }
@@ -227,6 +238,7 @@ loginPlus.Manager.prototype.init = function init(app,opts){
     var authenticateOptions={ 
         failureRedirect: opts.redirBaseUrl+opts.failedLoginUrlPath,
         failureFlash: true,
+        keepSessionInfo: true,
     }
     authenticateOptions[opts.successReturns?'successReturnToOrRedirect':'successRedirect'] = opts.redirBaseUrl+opts.successRedirect    
     app.post(opts.baseUrl+opts.loginUrlPath,
@@ -265,9 +277,11 @@ loginPlus.Manager.prototype.init = function init(app,opts){
                 req.body.newPassword, 
                 function(err, ok, flashMessage){
                     if(ok){
-                        req.logout();
-                        req.session.returnTo = null;
-                        res.redirect(opts.redirBaseUrl+opts.loginUrlPath);
+                        req.logout(function(err) {
+                            if (err) { return next(err); }
+                            req.session.returnTo = null;
+                            res.redirect(opts.redirBaseUrl+opts.loginUrlPath);
+                        });
                     }else{
                         if(err && opts.log.errors.getTime() > new Date().getTime()){
                             console.log('chpass error',err);

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "login-plus",
   "description": "login service for express",
-  "version": "1.6.3",
+  "version": "1.7.1",
   "author": "Codenautas <codenautas@googlegroups.com>",
   "license": "MIT",
   "repository": "codenautas/login-plus",
@@ -23,39 +23,39 @@
   "dependencies": {
     "connect-flash": "^0.1.1",
     "express-session": "^1.17.3",
-    "fs-extra": "^10.1.0",
+    "fs-extra": "^11.1.1",
     "passport": "^0.6.0",
     "passport-local": "^1.0.0",
     "session-file-store": "^1.5.0",
-    "best-globals": "^1.0.3",
-    "ensure-login": "^0.1.4",
-    "mini-tools": "^1.11.2",
+    "best-globals": "^1.1.0",
+    "ensure-login": "^0.1.5",
+    "mini-tools": "^1.12.0",
     "regexplicit": "^0.1.3"
   },
   "devDependencies": {
-    "body-parser": "^1.20.0",
+    "body-parser": "^1.20.2",
     "concat-stream": "^2.0.0",
     "cookie-parser": "^1.4.6",
     "expect.js": "^0.3.1",
-    "express": "^4.18.1",
+    "express": "^4.18.2",
     "nyc": "^15.1.0",
     "pug": "^3.0.2",
-    "mocha": "^10.0.0",
+    "mocha": "^10.2.0",
     "php-session": "^0.1.0",
     "read-yaml-promise": "^1.0.2",
-    "sinon": "^14.0.0",
-    "stylus": "^0.58.1",
-    "supertest": "^6.2.4",
+    "sinon": "^15.0.4",
+    "stylus": "^0.59.0",
+    "supertest": "^6.3.3",
     "ajax-best-promise": "^0.3.7",
-    "pg-promise-strict": "^1.2.6",
+    "pg-promise-strict": "^1.3.2",
     "self-explain": "^0.10.22",
-    "serve-content": "^0.3.17"
+    "serve-content": "^0.3.18"
   },
   "optionalDependencies": {
     "php-session": "^0.1.0"
   },
   "engines": {
-    "node": ">= 14"
+    "node": ">= 18"
   },
   "scripts": {
     "test": "mocha --reporter spec --bail --check-leaks --globals __core-js_shared__ --exit test/test*.js",