Extract redirect location handling and clarify behaviour.

Author: ioquatix

lib/async/http/relative_location.rb

@@ -16,7 +16,9 @@ module HTTP
 		class TooManyRedirects < StandardError
 		end
 
-		# A client wrapper which transparently handles both relative and absolute redirects to a given maximum number of hops.
+		# A client wrapper which transparently handles redirects to a given maximum number of hops.
+		#
+		# The default implementation will only follow relative locations (i.e. those without a scheme) and will only switch to GET if the original request was not a GET.
 		#
 		# The best reference for these semantics is defined by the [Fetch specification](https://fetch.spec.whatwg.org/#http-redirect-fetch).
 		#
@@ -58,6 +60,16 @@ def redirect_with_get?(request, response)
 				end
 			end
 
+			def handle_redirect(request, location)
+				uri = URI.parse(location)
+				
+				if uri.absolute?
+					return false
+				end
+				
+				request.path = Reference[request.path] + location
+			end
+			
 			def call(request)
 				# We don't want to follow redirects for HEAD requests:
 				return super if request.head?
@@ -83,12 +95,8 @@ def call(request)
 
 						response.finish
 
-						uri = URI.parse(location)
-						
-						if uri.absolute?
+						unless handle_redirect(request, location)
 							return response
-						else
-							request.path = Reference[request.path] + location
 						end
 
 						if request.method == GET or response.preserve_method?