limit the amount of redirects to a fixed number and also don't create…

… a new request object since it's already setup for us

motion/core/ns_url_request.rb

@@ -3,8 +3,10 @@ class NSURLRequest
   # Provides a to_s method so we can use inspect in instances and get
   # valuable information.
   def to_s
-    "#<#{self.class}:#{self.object_id} - url: #{self.URL.description}, cache policy: #{self.cachePolicy}, Pipelining: #{self.HTTPShouldUsePipelining}, main doc url: #{mainDocumentURL},\
- timeout: #{self.timeoutInterval}, network service type: #{self.networkServiceType} >"
+    "#<#{self.class}:#{self.object_id} - url: #{self.URL.description}, 
+    headers: #{self.allHTTPHeaderFields.inspect}, 
+    cache policy: #{self.cachePolicy}, Pipelining: #{self.HTTPShouldUsePipelining}, main doc url: #{mainDocumentURL},\
+    timeout: #{self.timeoutInterval}, network service type: #{self.networkServiceType} >"
   end
 
 end

motion/http.rb

@@ -159,13 +159,16 @@ def connection(connection, didReceiveData:received_data)
       def connection(connection, willSendRequest:request, redirectResponse:redirect_response)
         @redirection ||= 0
         @redirection += 1
-        log "##{@redirection} HTTP redirection: #{request} - #{self.description}"
-        new_request = request.mutableCopy
-        # new_request.setValue(@credentials.inspect, forHTTPHeaderField:'Authorization') # disabled while we figure this one out
-        new_request.setAllHTTPHeaderFields(@headers) if @headers
-        @connection.cancel
-        @connection = create_connection(new_request, self)
-        new_request
+        log "##{@redirection} HTTP redirection: #{request.inspect} - #{self.description}"
+        if @redirection >= 30
+          log "Too many redirections"
+          @response.error_message = "Too many redirections"
+          @request.done_loading!
+          call_delegator_with_response
+          nil
+        else
+          request
+        end
       end
 
       def connection(connection, didFailWithError: error)