@webqit/webflo 1.0.24 → 1.0.26

package/package.json

@@ -12,7 +12,7 @@
     "vanila-javascript"
   ],
   "homepage": "https://webqit.io/tooling/webflo",
-  "version": "1.0.24",
+  "version": "1.0.26",
   "license": "MIT",
   "repository": {
     "type": "git",

package/src/runtime-pi/WebfloStorage.js

@@ -8,6 +8,7 @@ export class WebfloStorage {
     #registry;
     #key;
     #store;
+    #modified = false;
 
     constructor(registry, key, request, session = null) {
         this.#registry = registry;
@@ -22,13 +23,12 @@ export class WebfloStorage {
         }
         if (!this.#store && !(this.#store = await this.#registry.get(this.#key))) {
             this.#store = {};
-            await this.#registry.set(this.#key, this.#store);
         }
         return this.#store;
     }
 
     async commit() {
-        if (!this.#store || !this.#key) return;
+        if (!this.#store || !this.#key || !this.#modified) return;
         await this.#registry.set(this.#key, this.#store);
     }
 
@@ -62,12 +62,14 @@ export class WebfloStorage {
 
     async set(key, value) {
         Reflect.set(await this.store(), key, value);
+        this.#modified = true;
         await this.emit(key, value);
         return this;
     }
 
     async delete(key) {
         Reflect.deleteProperty(await this.store(), key);
+        this.#modified = true;
         await this.emit(key);
         return this;
     }

package/src/runtime-pi/server/SessionStorage.js

@@ -44,7 +44,8 @@ export class SessionStorage extends WebfloStorage {
 
     async commit(response = null) {
         if (response && !response.headers.get('Set-Cookie', true).find((c) => c.name === '__sessid')) {
-            response.headers.append('Set-Cookie', `__sessid=${this.#sessionID}; Path=/; Secure; HttpOnly; SameSite=Lax; Max-Age=31536000`);
+            // expires six months
+            response.headers.append('Set-Cookie', `__sessid=${this.#sessionID}; Path=/; Secure; HttpOnly; SameSite=Lax; Max-Age=15768000`);
         }
         await super.commit();
     }

package/src/runtime-pi/server/WebfloServer.js

@@ -102,6 +102,7 @@ export class WebfloServer extends WebfloRuntime {
             }));
         }
         // ---------------
+        await this.setupCapabilities();
         this.control();
         if (this.#cx.logger) {
             if (this.#servers.size) {
@@ -202,10 +203,9 @@ export class WebfloServer extends WebfloRuntime {
         }
         if (this.#cx.server.capabilities?.redis && process.env[this.#cx.server.capabilities.redis_url_variable]) {
             const { Redis } = await import('ioredis');
-            this.#sdk.redis = !process.env[this.#cx.server.capabilities.redis_url_variable]
-                ? new Redis : new Redis(process.env[this.#cx.server.capabilities.redis_url_variable], {
-                    tls: { rejectUnauthorized: false }, // Required for Upstash
-                });
+            this.#sdk.redis = process.env[this.#cx.server.capabilities.redis_url_variable]
+                ? new Redis(process.env[this.#cx.server.capabilities.redis_url_variable])
+                : new Redis;
             console.log('Redis capabilities');
         }
         if (this.#cx.server.capabilities?.webpush) {
@@ -229,7 +229,6 @@ export class WebfloServer extends WebfloRuntime {
 
     #globalMessagingRegistry = new Map;
     async handleNodeWsRequest(wss, nodeRequest, socket, head) {
-        await this.setupCapabilities();
         const proto = this.getRequestProto(nodeRequest).replace('http', 'ws');
         const [fullUrl, requestInit] = this.parseNodeRequest(proto, nodeRequest, false);
         const scope = {};
@@ -260,7 +259,13 @@ export class WebfloServer extends WebfloRuntime {
         // Level 3 validation
         // and actual processing
         scope.request = this.createRequest(scope.url.href, requestInit);
-        scope.session = this.constructor.SessionStorage.create(scope.request, { secret: process.env[this.#cx.server.session_key_variable] });
+        scope.session = this.constructor.SessionStorage.create(scope.request, {
+            secret: process.env[this.#cx.server.session_key_variable],
+            registry: this.#sdk.redis && {
+                get: async (key) => { return JSON.parse(await this.#sdk.redis.get(key) || null) },
+                set: async (key, value) => { return await this.#sdk.redis.set(key, JSON.stringify(value), 'EX', 15768000) },
+            },
+        });
         if (!scope.error) {
             if (!(scope.clientMessagingRegistry = this.#globalMessagingRegistry.get(scope.session.sessionID))) {
                 scope.error = `Lost or invalid clientID`;
@@ -290,7 +295,6 @@ export class WebfloServer extends WebfloRuntime {
     }
 
     async handleNodeHttpRequest(nodeRequest, nodeResponse) {
-        await this.setupCapabilities();
         const proto = this.getRequestProto(nodeRequest);
         const [fullUrl, requestInit] = this.parseNodeRequest(proto, nodeRequest);
         const scope = {};
@@ -562,8 +566,8 @@ export class WebfloServer extends WebfloRuntime {
         scope.session = this.constructor.SessionStorage.create(scope.request, {
             secret: process.env[this.#cx.server.session_key_variable],
             registry: this.#sdk.redis && {
-                get: async (key) => { return await this.#sdk.redis.hgetall(key) },
-                set: async (key, value) => { return await this.#sdk.redis.hset(key, value) },
+                get: async (key) => { return JSON.parse(await this.#sdk.redis.get(key) || null) },
+                set: async (key, value) => { return await this.#sdk.redis.set(key, JSON.stringify(value), 'EX', 15768000) },
             },
         });
         const sessionID = scope.session.sessionID;