node-rtc-connection 1.0.14 → 1.0.16

package/dist/index.cjs

@@ -3,7 +3,7 @@
 var require$$0$1 = require('events');
 var require$$0 = require('dgram');
 var require$$3 = require('crypto');
-var require$$4 = require('os');
+var require$$5 = require('os');
 var require$$1 = require('net');
 
 function getDefaultExportFromCjs (x) {
@@ -1772,16 +1772,43 @@ function requireRTCIceTransport () {
 	      throw new TypeError('Candidate must be an object');
 	    }
 
-	    // Add to remote candidates list
+	    // Ensure candidate is properly formatted
+	    // If it's a plain object with a candidate string, parse it
+	    let parsedCandidate = candidate;
+	    const RTCIceCandidate = requireRTCIceCandidate();
+	    
+	    if (!(candidate instanceof RTCIceCandidate)) {
+	      // Try to create an RTCIceCandidate to parse and validate
+	      try {
+	        parsedCandidate = new RTCIceCandidate(candidate);
+	      } catch (err) {
+	        console.warn('Failed to parse remote candidate:', err.message);
+	        // Store the original anyway for compatibility
+	        this._remoteCandidates.push(candidate);
+	        return;
+	      }
+	    } else {
+	      parsedCandidate = candidate;
+	    }
+
+	    // Validate that candidate has required properties for connectivity checks
+	    if (!parsedCandidate.address || parsedCandidate.port === null || parsedCandidate.port === undefined) {
+	      console.warn('Remote candidate missing address or port, skipping connectivity checks');
+	      // Store the original candidate for compatibility
+	      this._remoteCandidates.push(candidate);
+	      return;
+	    }
+
+	    // Store the original candidate (for getRemoteCandidates compatibility)
 	    this._remoteCandidates.push(candidate);
 
 	    // Start connectivity checks if we're in checking state
 	    if (this._state === RTCIceTransportState.CHECKING && this._localCandidates.length > 0) {
-	      // Create pairs for this new remote candidate with all local candidates
+	      // Create pairs using the parsed candidate for connectivity checks
 	      for (const localCandidate of this._localCandidates) {
 	        const pair = {
 	          local: localCandidate,
-	          remote: candidate,
+	          remote: parsedCandidate,
 	          state: 'waiting'
 	        };
 	        this._candidatePairs.push(pair);
@@ -1897,7 +1924,7 @@ function requireRTCIceTransport () {
 	   * @private
 	   */
 	  async _gatherHostCandidates() {
-	    const os = require$$4;
+	    const os = require$$5;
 	    const interfaces = os.networkInterfaces();
 	    
 	    for (const [name, addrs] of Object.entries(interfaces)) {
@@ -2162,12 +2189,27 @@ function requireRTCIceTransport () {
 	   * @param {Object} pair - Candidate pair
 	   * @private
 	   */
+	  /**
+	   * Send ICE connectivity check to remote candidate
+	   * @param {Object} pair - Candidate pair
+	   * @private
+	   */
 	  _sendConnectivityCheck(pair) {
 	    const socket = this._sockets.get(pair.local.foundation);
 	    if (!socket || socket.type === 'turn') {
 	      return; // Skip TURN candidates for now
 	    }
 
+	    // Validate remote candidate has required properties
+	    if (!pair.remote.address || !pair.remote.port) {
+	      console.warn('Cannot send connectivity check: remote candidate missing address or port', {
+	        address: pair.remote.address,
+	        port: pair.remote.port,
+	        candidate: pair.remote.candidate
+	      });
+	      return;
+	    }
+
 	    const transactionId = crypto.randomBytes(12);
 	    const request = this._createBindingRequest(transactionId);
 	    
@@ -2178,7 +2220,7 @@ function requireRTCIceTransport () {
 	        }
 	      });
 	    } catch (err) {
-	      console.error(`Error sending connectivity check:`, err);
+	      console.error(`Error sending connectivity check to ${pair.remote.address || 'unknown'}:${pair.remote.port || 'unknown'}:`, err);
 	    }
 	  }
 
@@ -4787,7 +4829,7 @@ function requireRTCPeerConnection () {
 	    const { address, port } = await this._networkTransport.listen(0, '0.0.0.0');
 	    
 	    // Get local address (try to find non-localhost)
-	    const os = require$$4;
+	    const os = require$$5;
 	    const interfaces = os.networkInterfaces();
 	    let localAddress = '127.0.0.1';
 	    
@@ -5044,10 +5086,11 @@ function requireRTCPeerConnection () {
 	      if (this._sctpTransport.state === 'connected') {
 	        for (const channel of this._dataChannels.values()) {
 	          if (channel.readyState === 'connecting') {
-	            channel._setStateToOpen();
-	            
-	            // Hook up channel to network transport
+	            // Hook up channel to network transport first
 	            this._connectChannelToNetwork(channel);
+	            
+	            // Then set state to open (emits 'open' event)
+	            channel._setStateToOpen();
 	          }
 	        }
 	      }
@@ -5276,7 +5319,7 @@ function requireRTCPeerConnection () {
 	return RTCPeerConnection_1;
 }
 
-var version = "1.0.14";
+var version = "1.0.16";
 var require$$10 = {
 	version: version};