querysub 0.95.0 → 0.97.0

package/package.json

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

package/src/-c-identity/IdentityController.ts

@@ -176,11 +176,6 @@ const IdentityController = SocketFunction.register(
 );
 
 const changeIdentityOnce = cache(async function changeIdentityOnce(connectionId: { nodeId: string }) {
-    // Wait for time sync, otherwise the time in our change identity might be rejected.
-    await measureBlock(async () => {
-        await waitForFirstTimeSync();
-    }, "waitForFirstTimeSync");
-
     let nodeId = connectionId.nodeId;
     let threadKeyCert = getThreadKeyCert();
     let issuer = getIdentityCA();

package/src/-f-node-discovery/NodeDiscovery.ts

@@ -113,10 +113,17 @@ export function isOwnNodeId(nodeId: string): boolean {
 let nodeOverrides: string[] | undefined;
 let beforeGetNodeAllId = async () => { };
 export async function getAllNodeIds() {
+    await beforeGetNodeAllId();
+
+    // NOTE: We always wait for the time sync before returning any node ids. Because if we try to talk
+    //  to remote nodes with a massively out of sync time, they will likely reject our messages!
+    await measureBlock(async () => {
+        await waitForFirstTimeSync();
+    }, "waitForFirstTimeSync");
+
     if (nodeOverrides) {
         return nodeOverrides;
     }
-    await beforeGetNodeAllId();
     return Array.from(allNodeIds2);
 }
 
@@ -421,7 +428,7 @@ beforeGetNodeAllId = async () => {
     await discoveryReady.promise;
 };
 export async function onNodeDiscoveryReady() {
-    await beforeGetNodeAllId();
+    await getAllNodeIds();
 }
 if (isServer()) {
     setImmediate(() => {