cable_room 0.6.2.beta7 → 0.6.2.beta9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7309c58f5b1c28f703935326e40e13194a1e57c57ddd184cfa100ee7c23351d1
-  data.tar.gz: d666029a14ac41eb3b422edc7de2eaf1ce2cef8c95854f4116052c5bff8b447f
+  metadata.gz: 5a0b89cf59e8dc6fa9ebfed2efe728c0c2916bbac6d54738612944adec072762
+  data.tar.gz: 55db8ebfde7f3574755f8348e90a4986c8943600c9231b1ce0af723f94863f33
 SHA512:
-  metadata.gz: d252055f0b2e4062f01865a6f1ba482e555e3c8325eac88d29b0e0d34673aafccdab72fea916fda0fb6b36dbab4f8e7340ad120d90b463752d79d5c071dc9468
-  data.tar.gz: 2352b8d4315b7d3fff5389e6c6c9783b3314329cef0233500fd6a26fb91c8343f2244094ce0faa4a1b28f5e0868cef2932a27364ecfe2c9d7cfb66bb5377a2f4
+  metadata.gz: 60f838c24cc7b4bc43541c0cc9386f6a295824b0a44526a1fa0d018240966dc6c81665faed13edc76a7ae7fa4d0ab958dbe0c2486b85fc81f502e3fc819dafe2
+  data.tar.gz: 61c0ea11ca13892247ac210fe32cc2c1d5ee157f1da7125f4a9fdde22bee7e9d70a6468e539a7e4503814feefbcbdd80b2e2075291b4b48fd9cd8183e89d6415

data/lib/cable_room/room/port_management.rb

@@ -118,10 +118,7 @@ module CableRoom
       def handle_received_message(message)
         case message['type']
         when 'port_connected'
-          mo = @_port_clients[@current_message_origin]
-          return if mo
-
-          mo = @_port_clients[@current_message_origin] = PortClient.new(self, @current_message_origin)
+          mo = @_port_clients[@current_message_origin] ||= PortClient.new(self, @current_message_origin)
 
           mo.touch_activity!
           mo.tag!(message['tags'])

data/lib/cable_room/room_member.rb

@@ -51,7 +51,11 @@ module CableRoom
 
     def transmit_subscription_confirmation(...)
       super
-      _room_memberships.each(&:transmit_port_connected)
+      memberships = _room_memberships.to_a
+      memberships.each(&:streams_live!)
+      connection.worker_pool.async_exec(self, connection: connection) do
+        memberships.each(&:transmit_port_connected)
+      end
     end
   end
 
@@ -78,6 +82,7 @@ module CableRoom
 
       @has_left = false
       @has_established = false
+      @streams_live = false
 
       @cable_channel = cable_channel
       @room_class = room_class
@@ -146,6 +151,11 @@ module CableRoom
 
     def key; @room_key; end
 
+    # @internal
+    def streams_live!
+      @streams_live = true
+    end
+
     # @internal
     def transmit_port_connected
       msg = {
@@ -196,7 +206,10 @@ module CableRoom
         @has_established = true
         @on_joined&.call(self) if first_acknowledgement
       when 'room_opened'
-        transmit_port_connected
+        # Only safe once our own streams are confirmed; otherwise the room may acknowledge on the
+        # private port before anyone is listening. If we are not live yet, the subscription
+        # confirmation hook will announce for us.
+        transmit_port_connected if @streams_live
         @on_room_opened&.call(self)
       when 'room_closed'
         leave!
@@ -243,7 +256,9 @@ module CableRoom
 
         @preconfigure&.call(self)
 
-        # transmit_port_connected
+        # Announcing here races the async stream subscriptions: the room can acknowledge on the
+        # private port before it is live and the ack is lost. transmit_subscription_confirmation
+        # announces instead, once every stream is confirmed.
 
         maybe_provision_room
       end

data/lib/cable_room/version.rb

@@ -1,3 +1,3 @@
 module CableRoom
-  VERSION = "0.6.2.beta7".freeze
+  VERSION = "0.6.2.beta9".freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: cable_room
 version: !ruby/object:Gem::Version
-  version: 0.6.2.beta7
+  version: 0.6.2.beta9
 platform: ruby
 authors:
 - Ethan Knapp