@yz-social/webrtc 0.1.13 → 0.1.15

package/index.js

@@ -26,6 +26,13 @@ export class WebRTC {
     delete this.dataChannelOursPromises;
     delete this.dataChannelTheirsPromises;
   }
+  reportStates(label) {
+    const pc = this.pc;
+    const sctp = pc?.sctp;
+    const transport = sctp?.transport;
+    const iceTransport = transport?.iceTransport;
+    this.flog(label, 'connectionState:', pc?.connectionState, 'signalingState:', pc?.signalingState, 'iceConnectionState:', pc?.iceConnectionState, 'iceGatheringState:', pc?.iceGatheringState, 'transport state:', sctp?.state, transport?.state, iceTransport?.state, iceTransport?.gatheringState);
+  }
 
   // Number of instances at a time (if previous have been garbage collected), as of 1/27/26:
   static suggestedInstancesLimit =
@@ -46,9 +53,8 @@ export class WebRTC {
       if (!pc) return null;
       const state = pc.connectionState;
       if (state === 'connected') return this.signalsReadyResolver?.();
-      if (['new', 'connecting'].includes(state)) return null;
-      // closed, disconnected, failed: resolve this.closed promise.
-      this.log('connectionstatechange signaling/connection:', pc.signalingState, state);
+      if (['new', 'connecting', 'failed'].includes(state)) return null; // On failed, iceconnectionstatechange will restartIce.
+      // closed, disconnected: resolve this.closed promise.
       this.cleanup();
       return resolve(pc);
     };
@@ -75,8 +81,8 @@ export class WebRTC {
       case 'completed':
 	this._resolveIceCompleted?.(true);
 	break;
-      case 'failed':
-	this.pc.restartIce();
+      case 'failed': // This doesn't mean the connection is broken. It could just be looking for a better transport.
+	this.renegotiate();
 	break;
       default:
 	;
@@ -98,7 +104,7 @@ export class WebRTC {
     };
   }
   async renegotiate() { // Trigger negotiationneeded and promise to resolve when completed. Used in testing.
-    this._resolveIceCompleted?.(false); // clearing old promise, if any.
+    if (this._iceConnected) return this._iceConnected;
     const promise = this.iceConnected;
     this.pc.restartIce();
     return promise;
@@ -206,7 +212,8 @@ export class WebRTC {
     // This is used by a peer that is receiving signals in an out-of-band network request, and waiting for a response. (Compare transferSignals.)
     return this.responseSerializer = this.responseSerializer.then(async () => {
       this.log('respond', signals.length, 'signals');
-      const {promise, resolve} = Promise.withResolvers();
+      const {promise, resolve} = Promise.withResolvers(); // FIXME: this breaks Firefox.
+      //const {promise, resolve} = {}; // FIXME: tests "pass" on Firefox with this version.
       this.signalsReadyResolver = resolve;
       await this.onSignals(signals);
       if (!this.pendingSignals.length) await promise;
@@ -330,11 +337,12 @@ export class WebRTC {
       return;
     }
     const remote = stats.get(candidatePair.remoteCandidateId);
-    const {protocol, candidateType, address} = remote;
+    const {protocol, candidateType, address, port} = remote;
     const now = Date.now();
     const statsElapsed = now - this.connectionStartTime;
+    const local = stats.get(candidatePair.localCandidateId);
     Object.assign(this, {stats, transport, candidatePair, remote, protocol, candidateType, statsTime: now, statsElapsed});
-    if (doLogging) console.info(this.name, 'connected', protocol, candidateType, address, (statsElapsed/1e3).toFixed(1));
+    if (doLogging) console.info(new Date(), this.name, this.pc?.connectionState, protocol, candidateType, `${local.address}:${local.port}=>${address}:${port}`, (statsElapsed/1e3).toFixed(1));
   }
 
   static getPublicIP(stunServer = "stun:stun.l.google.com:19302") { // Promise external/WAN/public IP addresses for this device.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yz-social/webrtc",
-  "version": "0.1.13",
+  "version": "0.1.15",
   "description": "Streamlined portable webrtc management of p2p and client2server.",
   "keywords": [
     "webrtc",