ewelink 2.0.0 → 2.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 97046b44715a988f6ae92d986c0fad17ab4d5cae6420a125c10f23854cfaadc9
-  data.tar.gz: b95e52d88f8d57d5bf9f54b67d5a8c0779d26b66faadbc252e13eadb80854292
+  metadata.gz: b7b1ea698c178aeed55aa90970659bb230f84e611a1bff87d135e301a81bca0d
+  data.tar.gz: 73e86511d022f0481d0156b130ac08f679c5e06a1549ab1c903bb9b795187ee9
 SHA512:
-  metadata.gz: f153a280ff50b245434b2039cd3bb88341da6bd7688556a8a7c3fe4558a2b13a6be5c1497f5b06dc381495cb63d8c26d4d3f2625ea5bd4148e4c55a2f28b2faf
-  data.tar.gz: e3b69fa1607599325ea0f2867fbd0d32fd3abd56a4e7bd8d07107e083d4ebbfaf919bc4af34b28948163d1fb5340a53489aec1839cc59ba238f2917a8db03cc5
+  metadata.gz: 141e43b9082542fe3c6a598754b9d0e9e69f6b5612da8df2e56921e1b1df8c96517ac97059d782c7539177bf42889dbfd70b6f94186e99f8d22649071066ecf7
+  data.tar.gz: 445ce9af569e5b997eff1ff20fcafec709cc3edf0a60e278262ba66c56341cf389f6d133c4650f8916af995e42f61a132a56ba822455e8a6595003aca6e92a48

data/VERSION

@@ -1 +1 @@
-2.0.0
+2.2.2

data/ewelink.gemspec

@@ -16,8 +16,8 @@ Gem::Specification.new do |s|
   s.required_ruby_version = '>= 2.0.0'
 
   s.add_dependency 'activesupport', '>= 6.0.0', '< 7.0.0'
+  s.add_dependency 'faye-websocket', '>= 0.11.0', '< 0.12.0'
   s.add_dependency 'httparty', '>= 0.18.0', '< 0.19.0'
-  s.add_dependency 'websocket-client-simple', '>= 0.3.0', '< 0.4.0'
 
   s.add_development_dependency 'byebug', '>= 11.0.0', '< 12.0.0'
   s.add_development_dependency 'rake', '>= 12.0.0', '< 13.0.0'

data/lib/ewelink.rb

@@ -1,6 +1,8 @@
 require 'active_support'
 require 'active_support/core_ext'
 require 'byebug' if ENV['DEBUGGER']
+require 'eventmachine'
+require 'faye/websocket'
 require 'httparty'
 require 'io/console'
 require 'json'
@@ -9,7 +11,6 @@ require 'openssl'
 require 'optparse'
 require 'set'
 require 'timeout'
-require 'websocket-client-simple'
 
 module Ewelink
 

data/lib/ewelink/api.rb

@@ -5,12 +5,14 @@ module Ewelink
     APP_ID = 'oeVkj2lYFGnJu5XUtWisfW4utiN4u9Mq'
     APP_SECRET = '6Nz4n0xA8s8qdxQf2GqurZj2Fs55FUvM'
     DEFAULT_REGION = 'us'
+    REQUEST_TIMEOUT = 10.seconds
     RF_BRIDGE_DEVICE_UIID = 28
     SWITCH_DEVICES_UIIDS = [1, 5, 6, 24]
-    TIMEOUT = 10.seconds
     URL = 'https://#{region}-api.coolkit.cc:8080'
     UUID_NAMESPACE = 'e25750fb-3710-41af-b831-23224f4dd609';
     VERSION = 8
+    WEB_SOCKET_PING_TOLERANCE_FACTOR = 1.5
+    SWITCH_STATUS_CHANGE_CHECK_TIMEOUT = 2.seconds
     WEB_SOCKET_WAIT_INTERVAL = 0.2.seconds
 
     attr_reader :email, :password, :phone_number
@@ -20,15 +22,15 @@ module Ewelink
       @mutexs = {}
       @password = password.presence || raise(Error.new(":password must be specified"))
       @phone_number = phone_number.presence.try(:strip)
-      @switches_statuses = {}
       @web_socket_authenticated_api_keys = Set.new
+      @web_socket_switches_statuses = {}
       raise(Error.new(":email or :phone_number must be specified")) if email.blank? && phone_number.blank?
     end
 
     def press_rf_bridge_button!(uuid)
       synchronize(:press_rf_bridge_button) do
         button = find_rf_bridge_button!(uuid)
-        web_socket_wait_for(-> { web_socket_authenticated? }) do
+        web_socket_wait_for(-> { web_socket_authenticated? }, initialize_web_socket: true) do
           params = {
             'action' => 'update',
             'apikey' => button[:api_key],
@@ -41,6 +43,7 @@ module Ewelink
             'ts' => 0,
             'userAgent' => 'app',
           }
+          Ewelink.logger.debug(self.class.name) { "Pressing RF bridge button #{button[:uuid].inspect}" }
           send_to_web_socket(JSON.generate(params))
           true
         end
@@ -48,10 +51,42 @@ module Ewelink
     end
 
     def reload
-      Ewelink.logger.debug(self.class.name) { 'Reloading API (authentication token, devices, region,...)' }
-      dispose_web_socket
-      @switches_statuses.clear
-      [:@api_keys, :@authentication_token, :@devices, :@rf_bridge_buttons, :@region, :@switches, :@web_socket, :@web_socket_url].each do |variable|
+      Ewelink.logger.debug(self.class.name) { 'Reloading API (authentication token, devices, region, connections,...)' }
+
+      @web_socket_authenticated_api_keys.clear
+      @web_socket_switches_statuses.clear
+
+      [@web_socket_ping_thread, @web_socket_thread].each do |thread|
+        next unless thread
+        if Thread.current == thread
+          thread[:stop] = true
+        else
+          thread.kill
+        end
+      end
+
+      if @web_socket.present?
+        begin
+          @web_socket.close if @web_socket.open?
+        rescue
+          # Ignoring close errors
+        end
+      end
+
+      [
+        :@api_keys,
+        :@authentication_token,
+        :@devices,
+        :@last_web_socket_pong_at,
+        :@region,
+        :@rf_bridge_buttons,
+        :@switches,
+        :@web_socket_ping_interval,
+        :@web_socket_ping_thread,
+        :@web_socket_thread,
+        :@web_socket_url,
+        :@web_socket,
+      ].each do |variable|
         remove_instance_variable(variable) if instance_variable_defined?(variable)
       end
       self
@@ -93,21 +128,22 @@ module Ewelink
 
     def switch_on?(uuid)
       switch = find_switch!(uuid)
-      if @switches_statuses[switch[:uuid]].nil?
-        params = {
-          'action' => 'query',
-          'apikey' => switch[:api_key],
-          'deviceid' => switch[:device_id],
-          'sequence' => web_socket_sequence,
-          'ts' => 0,
-          'userAgent' => 'app',
-        }
-        web_socket_wait_for(-> { web_socket_authenticated? }) do
+      if @web_socket_switches_statuses[switch[:uuid]].nil?
+        web_socket_wait_for(-> { web_socket_authenticated? }, initialize_web_socket: true) do
+          Ewelink.logger.debug(self.class.name) { "Checking switch #{switch[:uuid].inspect} status" }
+          params = {
+            'action' => 'query',
+            'apikey' => switch[:api_key],
+            'deviceid' => switch[:device_id],
+            'sequence' => web_socket_sequence,
+            'ts' => 0,
+            'userAgent' => 'app',
+          }
           send_to_web_socket(JSON.generate(params))
         end
       end
-      web_socket_wait_for(-> { !@switches_statuses[switch[:uuid]].nil? }) do
-        @switches_statuses[switch[:uuid]] == 'on'
+      web_socket_wait_for(-> { !@web_socket_switches_statuses[switch[:uuid]].nil? }, initialize_web_socket: true) do
+        @web_socket_switches_statuses[switch[:uuid]] == 'on'
       end
     end
 
@@ -138,8 +174,8 @@ module Ewelink
         on = false
       end
       switch = find_switch!(uuid)
-      @switches_statuses[switch[:uuid]] = nil
-      web_socket_wait_for(-> { web_socket_authenticated? }) do
+      @web_socket_switches_statuses[switch[:uuid]] = nil
+      web_socket_wait_for(-> { web_socket_authenticated? }, initialize_web_socket: true) do
         params = {
           'action' => 'update',
           'apikey' => switch[:api_key],
@@ -151,9 +187,12 @@ module Ewelink
           'ts' => 0,
           'userAgent' => 'app',
         }
+        Ewelink.logger.debug(self.class.name) { "Turning switch #{switch[:uuid].inspect} #{on ? 'on' : 'off'}" }
         send_to_web_socket(JSON.generate(params))
-        true
       end
+      sleep(SWITCH_STATUS_CHANGE_CHECK_TIMEOUT)
+      switch_on?(switch[:uuid]) # Waiting for switch status update
+      true
     end
 
     private
@@ -164,6 +203,24 @@ module Ewelink
       end
     end
 
+    def authenticate_web_socket_api_keys
+      api_keys.each do |api_key|
+        params = {
+          'action' => 'userOnline',
+          'apikey' => api_key,
+          'appid' => APP_ID,
+          'at' => authentication_token,
+          'nonce' => nonce,
+          'sequence' => web_socket_sequence,
+          'ts' => Time.now.to_i,
+          'userAgent' => 'app',
+          'version' => VERSION,
+        }
+        Ewelink.logger.debug(self.class.name) { "Authenticating WebSocket API key: #{api_key.truncate(16).inspect}" }
+        send_to_web_socket(JSON.generate(params))
+      end
+    end
+
     def authentication_headers
       { 'Authorization' => "Bearer #{authentication_token}" }
     end
@@ -208,18 +265,6 @@ module Ewelink
       end
     end
 
-    def dispose_web_socket
-      @web_socket_authenticated_api_keys = Set.new
-      if instance_variable_defined?(:@web_socket)
-        begin
-          @web_socket.close if @web_socket.open?
-        rescue
-          # Ignoring close errors
-        end
-        remove_instance_variable(:@web_socket) if instance_variable_defined?(:@web_socket)
-      end
-    end
-
     def find_rf_bridge_button!(uuid)
       rf_bridge_buttons.find { |button| button[:uuid] == uuid } || raise(Error.new("No such RF bridge button with UUID: #{uuid.inspect}"))
     end
@@ -241,7 +286,7 @@ module Ewelink
       method = method.to_s.upcase
       headers = (options[:headers] || {}).reverse_merge('Content-Type' => 'application/json')
       Ewelink.logger.debug(self.class.name) { "#{method} #{url}" }
-      response = HTTParty.send(method.downcase, url, options.merge(headers: headers).reverse_merge(timeout: TIMEOUT))
+      response = HTTParty.send(method.downcase, url, options.merge(headers: headers).reverse_merge(timeout: REQUEST_TIMEOUT))
       raise(Error.new("#{method} #{url}: #{response.code}")) unless response.success?
       if response['error'] == 301 && response['region'].present?
         @region = response['region']
@@ -255,11 +300,29 @@ module Ewelink
       raise Error.new(e)
     end
 
-    def send_to_web_socket(data)
-      web_socket.send(data)
-    rescue
-      dispose_web_socket
-      raise
+    def send_to_web_socket(message)
+      if web_socket_outdated_ping?
+        Ewelink.logger.warn(self.class.name) { 'WebSocket ping is outdated' }
+        reload
+      end
+      web_socket.send(message)
+    rescue => e
+      reload
+      raise Error.new(e)
+    end
+
+    def start_web_socket_ping_thread(interval)
+      @last_web_socket_pong_at = Time.now
+      @web_socket_ping_interval = interval
+      Ewelink.logger.debug(self.class.name) { "Creating thread for WebSocket ping every #{@web_socket_ping_interval} seconds" }
+      @web_socket_ping_thread = Thread.new do
+        loop do
+          break if Thread.current[:stop]
+          sleep(@web_socket_ping_interval)
+          Ewelink.logger.debug(self.class.name) { 'Sending WebSocket ping' }
+          send_to_web_socket('ping')
+        end
+      end
     end
 
     def synchronize(name, &block)
@@ -268,69 +331,71 @@ module Ewelink
 
     def web_socket
       synchronize(:web_socket) do
-        @web_socket ||= begin
-          api = self
+        next @web_socket if @web_socket
 
-          WebSocket::Client::Simple.connect(web_socket_url) do |web_socket|
-            Ewelink.logger.debug(self.class.name) { "Opening WebSocket to #{web_socket_url}" }
+        @web_socket_thread = Thread.new do
+          EventMachine.run do
+            Ewelink.logger.debug(self.class.name) { "Opening WebSocket to #{web_socket_url.inspect}" }
 
-            web_socket.on(:close) do
-              api.instance_eval do
-                Ewelink.logger.debug(self.class.name) { 'WebSocket closed' }
-                dispose_web_socket
-              end
+            @web_socket = Faye::WebSocket::Client.new('wss://as-pconnect3.coolkit.cc:8080/api/ws')
+
+            @web_socket.on(:close) do |event|
+              Ewelink.logger.debug(self.class.name) { 'WebSocket closed' }
+              reload
             end
 
-            web_socket.on(:error) do |e|
-              api.instance_eval do
-                Ewelink.logger.warn(self.class.name) { "WebSocket error: #{e}" }
-                dispose_web_socket
-              end
+            @web_socket.on(:open) do |event|
+              Ewelink.logger.debug(self.class.name) { 'WebSocket opened' }
+              @last_web_socket_pong_at = Time.now
+              authenticate_web_socket_api_keys
             end
 
-            web_socket.on(:message) do |message|
-              api.instance_eval do
-                response = JSON.parse(message.data)
+            @web_socket.on(:message) do |event|
+              message = event.data
 
-                if response.key?('error') && response['error'] != 0
-                  Ewelink.logger.error(self.class.name) { "WebSocket message error: #{message.data}" }
-                  next
-                end
+              if message == 'pong'
+                Ewelink.logger.debug(self.class.name) { "Received WebSocket #{message.inspect} message" }
+                @last_web_socket_pong_at = Time.now
+                next
+              end
 
-                if response['apikey'].present? && !@web_socket_authenticated_api_keys.include?(response['apikey'])
-                  @web_socket_authenticated_api_keys << response['apikey']
-                  Ewelink.logger.debug(self.class.name) { "WebSocket successfully authenticated API key: #{response['apikey'].inspect}" }
-                end
+              begin
+                json = JSON.parse(message)
+              rescue => e
+                Ewelink.logger.error(self.class.name) { 'WebSocket JSON parse error' }
+                reload
+                next
+              end
 
-                if response['deviceid'].present? && response['params'].is_a?(Hash) && response['params']['switch'].present?
-                  switch = switches.find { |switch| switch[:device_id] == response['deviceid'] }
-                  @switches_statuses[switch[:uuid]] = response['params']['switch'] if switch.present?
-                end
+              if json.key?('error') && json['error'] != 0
+                Ewelink.logger.error(self.class.name) { "WebSocket message error: #{message.inspect}" }
+                reload
+                next
+              end
+
+              if !@web_socket_ping_thread && json.key?('config') && json['config']['hb'] == 1 && json['config']['hbInterval'].present?
+                start_web_socket_ping_thread(json['config']['hbInterval'] + 7)
               end
-            end
 
-            web_socket.on(:open) do
-              api.instance_eval do
-                Ewelink.logger.debug(self.class.name) { 'WebSocket opened' }
-                api_keys.each do |api_key|
-                  params = {
-                    'action' => 'userOnline',
-                    'apikey' => api_key,
-                    'appid' => APP_ID,
-                    'at' => authentication_token,
-                    'nonce' => nonce,
-                    'sequence' => web_socket_sequence,
-                    'ts' => Time.now.to_i,
-                    'userAgent' => 'app',
-                    'version' => VERSION,
-                  }
-                  Ewelink.logger.debug(self.class.name) { "Authenticating WebSocket API key: #{api_key.inspect}" }
-                  send_to_web_socket(JSON.generate(params))
+              if json['apikey'].present? && !@web_socket_authenticated_api_keys.include?(json['apikey'])
+                @web_socket_authenticated_api_keys << json['apikey']
+                Ewelink.logger.debug(self.class.name) { "WebSocket successfully authenticated API key: #{json['apikey'].truncate(16).inspect}" }
+              end
+
+              if json['deviceid'].present? && json['params'].is_a?(Hash) && json['params']['switch'].present?
+                switch = switches.find { |switch| switch[:device_id] == json['deviceid'] }
+                if switch.present?
+                  @web_socket_switches_statuses[switch[:uuid]] = json['params']['switch']
+                  Ewelink.logger.debug(self.class.name) { "Switch #{switch[:uuid].inspect} is #{@web_socket_switches_statuses[switch[:uuid]]}" }
                 end
               end
             end
           end
         end
+
+        web_socket_wait_for(-> { @web_socket.present? }) do
+          @web_socket
+        end
       end
     end
 
@@ -338,6 +403,10 @@ module Ewelink
       api_keys == @web_socket_authenticated_api_keys
     end
 
+    def web_socket_outdated_ping?
+      @last_web_socket_pong_at.present? && @web_socket_ping_interval.present? && @last_web_socket_pong_at < (@web_socket_ping_interval * WEB_SOCKET_PING_TOLERANCE_FACTOR).seconds.ago
+    end
+
     def web_socket_sequence
       (Time.now.to_f * 1000).round.to_s
     end
@@ -361,13 +430,18 @@ module Ewelink
       end
     end
 
-    def web_socket_wait_for(condition, &block)
-      web_socket # Initializes WebSocket
-      Timeout.timeout(TIMEOUT) do
-        loop do
-          return yield if condition.call
-          sleep(WEB_SOCKET_WAIT_INTERVAL)
+    def web_socket_wait_for(condition, initialize_web_socket: false, &block)
+      web_socket if initialize_web_socket
+      begin
+        Timeout.timeout(REQUEST_TIMEOUT) do
+          while !condition.call
+            sleep(WEB_SOCKET_WAIT_INTERVAL)
+          end
+          block_given? ? yield : true
         end
+      rescue => e
+        reload
+        raise Error.new(e)
       end
     end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ewelink
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.2.2
 platform: ruby
 authors:
 - Alexis Toulotte
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-09-02 00:00:00.000000000 Z
+date: 2020-09-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -31,45 +31,45 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 7.0.0
 - !ruby/object:Gem::Dependency
-  name: httparty
+  name: faye-websocket
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.18.0
+        version: 0.11.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: 0.19.0
+        version: 0.12.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.18.0
+        version: 0.11.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: 0.19.0
+        version: 0.12.0
 - !ruby/object:Gem::Dependency
-  name: websocket-client-simple
+  name: httparty
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.3.0
+        version: 0.18.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: 0.4.0
+        version: 0.19.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.3.0
+        version: 0.18.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: 0.4.0
+        version: 0.19.0
 - !ruby/object:Gem::Dependency
   name: byebug
   requirement: !ruby/object:Gem::Requirement