Merge remote-tracking branch 'origin/release4.7'

lib/right_http_connection.rb

@@ -418,13 +418,14 @@ def request(request_params, &block)
       current_params = @params.merge(request_params)
       exception = get_param(:exception, current_params) || RuntimeError
 
+      # Re-establish the connection if any of auth params has changed
+      same_auth_params_as_before = SECURITY_PARAMS.select do |param|
+        request_params[param] != get_param(param)
+      end.empty?
+
       # We save the offset here so that if we need to retry, we can return the file pointer to its initial position
       mypos = get_fileptr_offset(current_params)
       loop do
-        # Re-establish the connection if any of auth params has changed
-        same_auth_params_as_before = SECURITY_PARAMS.select do |param|
-          request_params[param] != get_param(param)
-        end.empty?
 
         current_params[:protocol] ||= (current_params[:port] == 443 ? 'https' : 'http')
         # (re)open connection to server if none exists or params has changed
@@ -454,8 +455,8 @@ def request(request_params, &block)
           unless @http          &&
                  @http.started? &&
                  same_server_as_before
-            start(current_params)
             same_auth_params_as_before = true
+            start(current_params)
           end
 
           # Detect if the body is a streamable object like a file or socket.  If so, stream that