Fix issues issue with HSTS upgrade redirects in firefox

RESTEDClient · Jul 27, 2017 · e5fddd2 · e5fddd2
1 parent 99f140f
commit e5fddd2
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/src/components/Response/Redirect.js b/src/components/Response/Redirect.js
@@ -45,7 +45,7 @@ function Redirect(props) {
         >
           {response.statusCode}
         </Status>
-        <small> {response.statusLine.replace(/.*\d{3} /, '')}</small>
+        <small> {response.statusLine && response.statusLine.replace(/.*\d{3} /, '')}</small>
       </h3>
 
       <Headers expanded headers={headers} />

diff --git a/src/propTypes/redirect.js b/src/propTypes/redirect.js
@@ -7,7 +7,6 @@ export const redirectShape = {
   documentUrl: string.isRequired,
   frameId: number.isRequired,
   fromCache: bool.isRequired,
-  ip: string.isRequired,
   method: oneOf(REQUEST_METHODS).isRequired,
   originUrl: string.isRequired,
   parentFrameId: number.isRequired,
@@ -19,13 +18,15 @@ export const redirectShape = {
       value: string.isRequired,
     }).isRequired,
   ).isRequired,
-  statusCode: number.isRequired,
-  statusLine: string.isRequired,
   tabId: number.isRequired,
   time: number.isRequired,
   timeStamp: number.isRequired,
   type: string.isRequired,
   url: string.isRequired,
+  // Not present on HSTS upgrade requests in Firefox
+  ip: string,
+  statusCode: number,
+  statusLine: string,
 };
 
 export default shape(redirectShape);