Added resave to fix issues of session expiry (#30)

* Added resave to fix session expiry issues * Added touch instead of resave * Added test for accessing multiple sites
rorylshanks · Mar 19, 2024 · f62adae · f62adae
1 parent cfa0db9
commit f62adae
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/lib/sso.js b/lib/sso.js
@@ -210,6 +210,7 @@ async function redirectToSsoProvider(req, res) {
         var redirectBasePath = getRedirectBasepath()
 
         if (req.session.loggedin) {
+            req.session.touch()
             let jwtPayload = {
                 protocol: redirectToken.protocol,
                 host: redirectToken.host,

diff --git a/test/e2e/tests/basic_test.js b/test/e2e/tests/basic_test.js
@@ -86,4 +86,12 @@ Scenario('Dynamic Upstreams Test SRV', async ({ I }) => {
 Scenario('Unauthenticated Access test', async ({ I }) => {
     I.amOnPage('http://test-unauthenticated-access.localtest.me/get');
     I.see("x-public-access")
-});
+});
+
+Scenario('Multiple site access without login', async ({ I }) => {
+    I.amOnPage('http://test-basic-login.localtest.me/get');
+    I.login();
+    I.see("x-veriflow-user-id")
+    I.amOnPage('http://test-https-upstream.localtest.me/get');
+    I.see("x-veriflow-user-id")
+});