Add target on log for proxies, even if not coming for redirectionio agent

Author: joelwurtz

src/Client.php

@@ -78,15 +78,17 @@ public function findRedirect(Request $request)
         }
 
         $ruleId = null;
+        $location = null;
+
         if (isset($json['matched_rule'], $json['matched_rule']['id'])) {
             $ruleId = $json['matched_rule']['id'];
         }
 
-        if (410 === $json['status_code']) {
-            return new Response(410, $ruleId);
+        if (isset($json['location'])) {
+            $location = $json['location'];
         }
 
-        return new RedirectResponse($json['location'], (int) $json['status_code'], $ruleId);
+        return new Response((int) $json['status_code'], $ruleId, $location);
     }
 
     public function log(Request $request, Response $response)
@@ -101,7 +103,7 @@ public function log(Request $request, Response $response)
             'use_json' => true,
         ];
 
-        if ($response instanceof RedirectResponse) {
+        if ($response->getLocation()) {
             $responseContext['target'] = $response->getLocation();
         }
 

src/HttpMessage/RedirectResponse.php

@@ -6,15 +6,18 @@ class Response
 {
     private $statusCode;
     private $ruleId;
+    private $location;
 
     /**
      * @param int    $statusCode
      * @param string $ruleId
+     * @param string $location
      */
-    public function __construct($statusCode = 200, $ruleId = null)
+    public function __construct($statusCode = 200, $ruleId = null, $location = null)
     {
         $this->statusCode = $statusCode;
         $this->ruleId = $ruleId;
+        $this->location = $location;
     }
 
     public function getStatusCode()
@@ -26,4 +29,9 @@ public function getRuleId()
     {
         return $this->ruleId;
     }
+
+    public function getLocation()
+    {
+        return $this->location;
+    }
 }