teleportxr 1.0.76 → 1.0.78

package/client/geometry_service.js

@@ -255,7 +255,12 @@ class GeometryService {
 		}
 
 		var node = this.scene.GetNode(node_uid);
-		console.log("Adding node ", node.name, " for client ", this.clientID);
+		if(diff>0 && !already_present)
+			console.log("Adding node ", node.name, " for client ", this.clientID);
+		else if(diff<0 && already_present)
+			console.log("Removing node ", node.name, " for client ", this.clientID);
+		else
+			console.log("Changing node ", node.name, " count from ", old_count, " to ", old_count+diff, " for client ", this.clientID);
 		this.streamedNodes.set(node_uid, old_count + diff);
 		var meshResources = [];
 		node.components.forEach((component) => {

package/connections/webrtcconnection.js

@@ -121,6 +121,10 @@ class WebRtcConnection extends EventEmitter
 			this.peerConnection = null;
 		}
 		this.peerConnection		=new DefaultRTCPeerConnection({ sdpSemantics: 'unified-plan', iceServers: this.iceServers, iceTransportPolicy: this.iceTransportPolicy});
+		// New PeerConnection means new data channels — the "all channels open"
+		// callback must be allowed to fire again. Without this reset the latch from
+		// the previous PeerConnection's lifetime suppresses UpdateStreaming kick.
+		this._dataChannelsOpenFired = false;
 		this.beforeOffer();
 		this.connectionTimer = this.options.setTimeout(() =>
 		{

package/package.json

@@ -16,7 +16,7 @@
   },
   "name": "teleportxr",
   "description": "Teleport Spatial Server on node.js",
-  "version": "1.0.76",
+  "version": "1.0.78",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/simul/teleport-nodejs.git"

package/signaling.js

@@ -133,14 +133,20 @@ function processInitialRequest(clientID, signalingClient, content) {
 		// Apparently the CLIENT thinks they've disconnected.
 		// The client might, as far as we know, have lost the information it needs to continue the connection.
 		// Therefore we should resend everything required.
-		//signalingClient.ChangeSignalingState(SignalingState.STREAMING);
 		console.log(
 			"Warning: Client " +
 				clientID +
 				" reconnected, but we didn't know we'd lost them."
 		);
-		// It may be just that the connection request was already in flight when we accepted its predecessor.
-		//sendResponseToClient(clientID);
+		// Tear down any stale Client + WebRtcConnection from the previous incarnation.
+		// Without this the next handshake's StartStreaming() races against the old
+		// peer connection's still-open data channels, and the reliable channel the
+		// server publishes commands to ends up being the wrong one — Origin/Lighting
+		// acks never come back and the server eventually gives up.
+		disconnectClient(clientID);
+		if (webRtcConnectionManager)
+			webRtcConnectionManager.destroyConnection(clientID);
+		signalingClient.ChangeSignalingState(SignalingState.START);
 		startStreaming(signalingClient);
 		return;
 	}