tweak http request header (#176)

* tweak http request header

* tweak log level and gem-update

Author: KazuCocoa

.travis.yml

@@ -9,7 +9,6 @@ rvm:
 #  - 2.6
 
 before_install:
-  - gem update --system
   - gem update bundler
 
 script:

CHANGELOG.md

@@ -4,6 +4,7 @@ Read `release_notes.md` for commit level details.
 
 ## [Unreleased]
 ### Enhancements
+- Tweak header like `appium/ruby_lib_core/2.2.1 (selenium/3.141.0 (ruby macosx))`
 
 ### Bug fixes
 

lib/appium_lib_core/common/base/http_default.rb

@@ -8,8 +8,30 @@ class Default < Selenium::WebDriver::Remote::Http::Default
           DEFAULT_HEADERS = {
             'Accept' => CONTENT_TYPE,
             'Content-Type' => "#{CONTENT_TYPE}; charset=UTF-8",
-            'User-Agent' => "appium/ruby_lib_core/#{VERSION}"
+            'User-Agent' =>
+              "appium/ruby_lib_core/#{VERSION} (#{::Selenium::WebDriver::Remote::Http::Common::DEFAULT_HEADERS['User-Agent']})"
           }.freeze
+
+          # override to use default header
+          # https://github.com/SeleniumHQ/selenium/blob/master/rb/lib/selenium/webdriver/remote/http/common.rb#L46
+          def call(verb, url, command_hash)
+            url      = server_url.merge(url) unless url.is_a?(URI)
+            headers  = DEFAULT_HEADERS.dup
+            headers['Cache-Control'] = 'no-cache' if verb == :get
+
+            if command_hash
+              payload                   = JSON.generate(command_hash)
+              headers['Content-Length'] = payload.bytesize.to_s if [:post, :put].include?(verb)
+
+              Logger.info("   >>> #{url} | #{payload}")
+              Logger.debug("     > #{headers.inspect}")
+            elsif verb == :post
+              payload = '{}'
+              headers['Content-Length'] = '2'
+            end
+
+            request verb, url, headers, payload
+          end
         end
       end
     end

test/test_helper.rb

@@ -8,7 +8,7 @@
 require 'minitest/reporters'
 require 'minitest'
 
-Appium::Logger.level = ::Logger::FATAL # Show Logger logs only they are fatal
+Appium::Logger.level = ::Logger::ERROR # Show Logger logs only they are error
 
 Minitest::Reporters.use! Minitest::Reporters::ProgressReporter.new