#303: do not logout if token refresh failed due to 5xx error

src/components/user/middlewares/refreshTokenMiddleware.js

@@ -68,16 +68,19 @@ export default function refreshTokenMiddleware({dispatch, getState}) {
 }
 
 function requestAccessToken(refreshToken, dispatch) {
-    let promise;
     if (refreshToken) {
-        promise = authentication.requestToken(refreshToken);
+        return authentication.requestToken(refreshToken)
-    } else {
+            .then(({token}) => dispatch(updateToken(token)))
-        promise = Promise.reject();
+            .catch((resp = {}) => {
+                if (resp.originalResponse && resp.originalResponse.status >= 500) {
+                    return Promise.reject(resp);
+                }
+
+                return dispatch(logoutAll());
+            });
     }
 
-    return promise
+    return dispatch(logoutAll());
-        .then(({token}) => dispatch(updateToken(token)))
-        .catch(() => dispatch(logoutAll()));
 }
 
 

tests/components/user/middlewares/refreshTokenMiddleware.test.js

@@ -110,7 +110,7 @@ describe('refreshTokenMiddleware', () => {
                 );
             });
 
-            it('should if token can not be parsed', () => {
+            it('should logout if token can not be parsed', () => {
                 const account = {
                     token: 'realy bad token',
                     refreshToken
@@ -143,6 +143,22 @@ describe('refreshTokenMiddleware', () => {
                     ])
                 );
             });
+
+            it('should not logout if request failed with 5xx', () => {
+                const resp = {
+                    originalResponse: {
+                        status: 500
+                    }
+                };
+
+                authentication.requestToken.returns(Promise.reject(resp));
+
+                return expect(middleware.before({url: 'foo', options: {}}), 'to be rejected with', resp).then(() =>
+                    expect(dispatch, 'to have no calls satisfying', [
+                        {payload: {isGuest: true}}
+                    ])
+                );
+            });
         });
 
         it('should not be applied if no token', () => {