querysub 0.268.0 → 0.270.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "querysub",
-  "version": "0.268.0",
+  "version": "0.270.0",
   "main": "index.js",
   "license": "MIT",
   "note1": "note on node-forge fork, see https://github.com/digitalbazaar/forge/issues/744 for details",

package/src/3-path-functions/pathFunctionLoader.ts

@@ -323,6 +323,7 @@ export function getSpecFromModule(module: NodeJS.Module): LoadFunctionSpec | und
 async function getModuleFromSpecBase(
     spec: LoadFunctionSpec
 ): Promise<NodeJS.Module> {
+    console.log(blue(`Loading module for ${JSON.stringify(spec)}`));
     // Register ourself as overlapping, to prevent ambient modules from being set in this way.
     registerSpec(module.filename, "overlapping");
 

package/src/4-deploy/edgeBootstrap.ts

@@ -247,11 +247,17 @@ async function edgeNodeFunction(config: {
     progressUI.setMessage("Finding available server...");
     let retryCount = 0;
     while (true) {
+        let cached = !!cachedConfig;
         try {
             let booted = await tryToBoot();
             if (booted) break;
         } catch (e) {
             console.error(e);
+            // If it's a cached failure, it might be that the cache is out of date, so rerun immediately without waiting or an error message.
+            if (cached) {
+                cachedConfig = undefined;
+                continue;
+            }
         }
         retryCount++;
         progressUI.setMessage(`No available servers, retried ${retryCount++} times`);
@@ -290,6 +296,7 @@ async function edgeNodeFunction(config: {
     }
     async function getEdgeNodeConfig(): Promise<EdgeNodeConfig> {
         let edgeIndex = cachedConfig || await (await fetch(config.edgeNodeConfigURL)).json() as EdgeNodesIndex;
+        cachedConfig = undefined;
 
         edgeIndex.edgeNodes.sort((a, b) => -(a.bootTime - b.bootTime));
         // Filter out duplicate hostnames (taking the max bootTime). We can't have multiple services using the same port, so duplicates are DEFINITELY dead. This fixes some issues with accessing old services, and generally makes reading the logs easier.
@@ -300,7 +307,6 @@ async function edgeNodeFunction(config: {
             return true;
         });
 
-        cachedConfig = undefined;
         let liveHashForced = false;
         if (Date.now() < liveHashOverrideExpiryTime) {
             liveHashForced = true;

package/src/4-querysub/Querysub.ts

@@ -801,6 +801,8 @@ export class Querysub {
             let existingRecords = await getRecords("A", getDomain());
             if (ip !== "127.0.0.1") {
                 let validRecords: string[] = [];
+                // Ignore ourself, we want OTHER records.
+                existingRecords = existingRecords.filter(x => x !== ip);
                 await Promise.all(existingRecords.map(async (record) => {
                     let isListening = await timeoutToUndefined(timeInSecond * 10, testTCPIsListening(record, 443));
                     if (isListening) {

package/src/deployManager/spec.txt

@@ -1,8 +1,3 @@
-8) Fix deploy user notification issue, where the refresh button doesn't work?
-    - For application level updates
-    - Just an application commit, then deploy the service, then the refresh button should show up, and work
-
-
 8) Use a special service for the HTTP bootstrapper, and then have 2 others that are on other ports
     --bootstraponly is added, and should work?
 

package/src/library-components/SyncedController.ts

@@ -95,6 +95,8 @@ export function getSyncedController<T extends SocketRegistered>(
          */
         reads?: { [key in keyof T["nodes"][""]]?: string[]; };
         writes?: { [key in keyof T["nodes"][""]]?: string[]; };
+        // TODO: Add a flag to give timeouts, so values are automatically refreshed every once in a while, so pages don't get too stale?
+        //  - BUT with a "softRefresh", so we don't trigger a re-render, we only update it if it is accessed again!
     }
 ): {
     (nodeId: string): {