New timeout system for packager (codesandbox#17)

* Check for status code 408 when retrying

* Change status code to 504

Author: CompuIves

src/app/store/api/actions.js

@@ -23,10 +23,10 @@ export function createAPIActions(prefix: string, suffix: string): APIActions {
   };
 }
 
-const getMessage = (error: Error) => {
+const getMessage = (error: Error & { response: ?Object }) => {
   const response = error.response;
 
-  if (response.status >= 500) {
+  if (!response || response.status >= 500) {
     sendError(error);
   }
 
@@ -40,6 +40,7 @@ const getMessage = (error: Error) => {
       return errors;
     }
   }
+
   return error.message;
 };
 

src/app/store/entities/sandboxes/bundler/index.js

@@ -11,6 +11,8 @@ const debug = _debug('cs:app:packager');
 
 export const PACKAGER_URL = 'https://cdn.jsdelivr.net/webpack/v3';
 
+const RETRY_COUNT = 10;
+
 /**
  * Request the packager, if retries > 4 it will throw if something goes wrong
  * otherwise it will retry again with an incremented retry
@@ -28,7 +30,10 @@ async function requestPackager(query: string) {
 
       return { ...result, url };
     } catch (e) {
-      if (retries < 5) {
+      const statusCode = e.response && e.response.status;
+
+      // 504 status code means the bundler is still bundling
+      if (retries < RETRY_COUNT && statusCode === 504) {
         retries += 1;
       } else {
         throw e;