socket-function 1.1.1 → 1.1.3

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "socket-function",
-	"version": "1.1.1",
+	"version": "1.1.3",
 	"main": "index.js",
 	"license": "MIT",
 	"dependencies": {

package/src/Zip.ts

@@ -10,7 +10,7 @@ setFlag(require, "pako", "allowclient", true);
 const SYNC_THRESHOLD_BYTES = 100_000_000;
 const ZIP_SYNC_THRESHOLD_BYTES = 10_000_000;
 
-// IMPORTANT! If this ever profiles as being slow, we should switch to a wasm implementation of L4Z. For our highly repetitive data, it's almost as efficient in terms of size, and it should be significantly more efficient in terms of time.
+// IMPORTANT! If this is slow, add a LZ4 version, which uses lz4-wasm-nodejs. This should be about 10X faster in both dcompression and decompression, only using 50% more space (which isn't great, but... if it is lagging the main thread it might be worth it).
 export class Zip {
     @measureFnc
     public static async gzip(buffer: Buffer, level?: number): Promise<Buffer> {

package/src/nodeCache.ts

@@ -49,6 +49,8 @@ export function getNodeIdLocation(nodeId: string): { address: string, port: numb
     return { address, port: parseInt(port) };
 }
 
+// NOTE: This returns the full domain.
+//  - It didn't used to, but I think it also wasn't used before, and now we're using this to find the ip. 
 export function getNodeIdDomain(nodeId: string): string {
     let result = getNodeIdDomainMaybeUndefined(nodeId);
     if (result === undefined) {
@@ -61,7 +63,7 @@ export function getNodeIdDomainMaybeUndefined(nodeId: string): string | undefine
     if (!location) {
         return undefined;
     }
-    return new URL(location.address).hostname.split(".").slice(-2).join(".");
+    return new URL("https://" + location.address).hostname;
 }
 
 // NOTE: CallFactory turns into an actual CallFactory when registerNodeClient is called