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

Author: joelwurtz

package.json

@@ -37,6 +37,7 @@
     "babel-runtime": "^6.26.0"
   },
   "devDependencies": {
+    "@redirectionio/proxy": "file:./.",
     "babel-cli": "^6.26.0",
     "babel-eslint": "^8.2.1",
     "babel-plugin-transform-runtime": "^6.23.0",
@@ -45,7 +46,6 @@
     "eslint": "^4.14.0",
     "express": "^4.16.2",
     "jest": "^22.0.4",
-    "pm2": "^2.9.3",
-    "@redirectionio/proxy": "file:./"
+    "pm2": "^2.9.3"
   }
 }

src/Client.js

@@ -4,7 +4,6 @@ import AgentNotFoundError from './Error/AgentNotFoundError'
 import BadConfigurationError from './Error/BadConfigurationError'
 import ConnectionNotWorkingError from './Error/ConnectionNotWorkingError'
 import NullLogger from './Logger/NullLogger'
-import RedirectResponse from './HttpMessage/RedirectResponse'
 import Response from './HttpMessage/Response'
 
 export default class Client {
@@ -85,11 +84,7 @@ export default class Client {
             ruleId = response.matched_rule.id
         }
 
-        if (410 === response.status_code) {
-            return new Response(410, ruleId)
-        }
-
-        return new RedirectResponse(response.location, Number(response.status_code), ruleId)
+        return new Response(Number(response.status_code), ruleId, response.location)
     }
 
     /**
@@ -107,7 +102,7 @@ export default class Client {
             'use_json': true,
         }
 
-        if (response instanceof RedirectResponse) {
+        if (response.location) {
             context.target = response.location
         }
 

src/HttpMessage/RedirectResponse.js

@@ -1,6 +1,7 @@
 export default class Response {
-    constructor(statusCode = 200, ruleId = null) {
+    constructor(statusCode = 200, ruleId = null, location = null) {
         this.statusCode = statusCode
         this.ruleId = ruleId
+        this.location = location
     }
 }