Fixed request_attach, set attaching state first, then send attach pro…

…to msg
ably · Jul 3, 2024 · 0af9002 · 0af9002
1 parent ce628e7
commit 0af9002
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/ably/realtime/channel/channel_manager.rb b/lib/ably/realtime/channel/channel_manager.rb
@@ -52,10 +52,10 @@ def log_channel_error(error)
       # @param reason
       # @option options [Ably::Models::ErrorInfo]  :reason
       def request_reattach(reason = nil)
-        send_attach_protocol_message
-        logger.debug { "Explicit channel reattach request sent to Ably due to #{reason}" }
         channel.set_channel_error_reason(reason) if reason
         channel.transition_state_machine! :attaching, reason: reason unless channel.attaching?
+        send_attach_protocol_message
+        logger.debug { "Explicit channel reattach request sent to Ably due to #{reason}" }
       end
 
       def duplicate_attached_received(protocol_message)