Updated docs

Fixed some lining errors
Added missing test dependency

Author: dodsky

.babelrc

@@ -1,8 +1,8 @@
 {
-  presets: ["es2015"],
-  plugins: ["transform-object-rest-spread",
+  "presets": ["es2015"],
+  "plugins": ["transform-object-rest-spread",
     ["babel-plugin-transform-builtin-extend", {
-      globals: ["Error", "Array"]
+      "globals": ["Error", "Array"]
     }]],
-  sourceMaps: true
+  "sourceMaps": true
 }

README.md

@@ -6,6 +6,7 @@ by renewing the current access token and then retrying an initial fetch operatio
 If you are not familiar with refresh token flow you should check some of the following resources:
 - [RFC standards track regarding refresh token flow](https://tools.ietf.org/html/rfc6749#page-10)
 - [Auth0 blog - Refresh Tokens: When to Use Them and How They Interact with JWTs](https://auth0.com/blog/refresh-tokens-what-are-they-and-when-to-use-them/)
+- [Shoutem blog - Keeping your tokens fresh](https://medium.com/shoutem/keeping-your-api-tokens-fresh-72059a7b0586)
 
 >Note:
 This library expects that fetch and promise api's are available at target environment. You should
@@ -46,7 +47,7 @@ config: {
   
   // Checks if response should be considered unauthorized (by default only 401 responses are 
   // considered unauthorized). Override this method if you need to trigger token renewal for 
-  // other response statuses.
+  // other response statuses. Check API reference for helper method which defines default behaviour
   isResponseUnauthorized: (response) => boolean,
    
   // (Required) Adds authorization for intercepted requests
@@ -97,7 +98,7 @@ to stop fetch interception.
    ...
 ```
 
-## API reference
+## API reference <a name="api-reference"></a>
 
 ### Exports
  `configure(configuration)`
@@ -112,6 +113,14 @@ to stop fetch interception.
 
  Clears all tokens from interceptor.
 
+ `isResponseUnauthorized(response)`
+ 
+ Utility method which determines if given response should be considered unauthorized. 
+ By default, responses with status code `401` are considered unauthorized.
+ You can use this method in `isResponseUnauthorized` of `config` object 
+ when you want to extend default behaviour.
+ 
+ 
 ## Tests
 
 ```

package.json

@@ -34,6 +34,7 @@
     "babel-core": "^6.9.1",
     "babel-eslint": "^6.0.0",
     "babel-plugin-transform-builtin-extend": "^1.1.2",
+    "babel-plugin-transform-object-rest-spread": "^6.23.0",
     "babel-preset-es2015": "^6.9.0",
     "babel-preset-stage-0": "^6.3.13",
     "babel-register": "^6.9.0",

src/FetchInterceptor.js

@@ -195,7 +195,7 @@ export default class FetchInterceptor {
       fetchArgs,
       fetchResolve,
       fetchReject,
-    }
+    };
   }
 
   createRequest(requestContext) {
@@ -205,14 +205,13 @@ export default class FetchInterceptor {
     return {
       ...requestContext,
       request,
-    }
+    };
   }
 
   shouldIntercept(requestContext) {
     const { request } = requestContext;
-    const { shouldIntercept } = this.config;
 
-    return Promise.resolve(shouldIntercept(request))
+    return Promise.resolve(this.config.shouldIntercept(request))
       .then(shouldIntercept =>
         ({ ...requestContext, shouldIntercept })
       );
@@ -229,10 +228,10 @@ export default class FetchInterceptor {
     const { accessToken } = this.accessTokenProvider.getAuthorization();
     const { authorizeRequest } = this.config;
 
-    if (request && accessToken){
+    if (request && accessToken) {
       return Promise.resolve(authorizeRequest(request, accessToken))
-        .then(request =>
-          ({ ...requestContext, accessToken, request })
+        .then(authorizedRequest =>
+          ({ ...requestContext, accessToken, request: authorizedRequest })
         );
     }
 
@@ -241,14 +240,13 @@ export default class FetchInterceptor {
 
   shouldFetch(requestContext) {
     const { request } = requestContext;
-    const { shouldFetch } = this.config;
 
     // verifies all outside conditions from config are met
-    if (!shouldFetch) {
+    if (!this.config.shouldFetch) {
       return requestContext;
     }
 
-    return Promise.resolve(shouldFetch(request))
+    return Promise.resolve(this.config.shouldFetch(request))
       .then(shouldFetch =>
         ({ ...requestContext, shouldFetch })
       );
@@ -282,15 +280,14 @@ export default class FetchInterceptor {
 
   shouldInvalidateAccessToken(requestContext) {
     const { shouldIntercept } = requestContext;
-    const { shouldInvalidateAccessToken } = this.config;
 
     if (!shouldIntercept) {
       return requestContext;
     }
 
     const { response } = requestContext;
     // check if response invalidates access token
-    return Promise.resolve(shouldInvalidateAccessToken(response))
+    return Promise.resolve(this.config.shouldInvalidateAccessToken(response))
       .then(shouldInvalidateAccessToken =>
         ({ ...requestContext, shouldInvalidateAccessToken })
       );
@@ -319,12 +316,11 @@ export default class FetchInterceptor {
 
     // can only be empty on network errors
     if (!response) {
-      fetchReject();
-      return;
+      return fetchReject();
     }
 
     if (shouldIntercept && isResponseUnauthorized(response)) {
-      throw new TokenExpiredException({ ...requestContext })
+      throw new TokenExpiredException({ ...requestContext });
     }
 
     if (this.config.onResponse) {

src/index.js

@@ -4,24 +4,11 @@ import {
   isWeb,
   isNode,
 } from './services/environment';
+import { isResponseUnauthorized } from './services/http';
 import FetchInterceptor from './FetchInterceptor';
 
 let interceptor = null;
 
-function init() {
-  if (isReactNative()) {
-    attach(global);
-  } else if (isWorker()) {
-    attach(self);
-  } else if (isWeb()) {
-    attach(window);
-  } else if (isNode()) {
-    attach(global);
-  } else {
-    throw new Error('Unsupported environment for fetch-token-intercept');
-  }
-}
-
 export function attach(env) {
   if (!env.fetch) {
     throw Error('No fetch available. Unable to register fetch-token-intercept');
@@ -35,10 +22,26 @@ export function attach(env) {
   interceptor = new FetchInterceptor(env.fetch);
 
   // monkey patch fetch
+  // eslint-disable-next-line no-unused-vars
   const fetchWrapper = fetch => (...args) => interceptor.intercept(...args);
+  // eslint-disable-next-line no-param-reassign
   env.fetch = fetchWrapper(env.fetch);
 }
 
+function init() {
+  if (isReactNative()) {
+    attach(global);
+  } else if (isWorker()) {
+    attach(self);
+  } else if (isWeb()) {
+    attach(window);
+  } else if (isNode()) {
+    attach(global);
+  } else {
+    throw new Error('Unsupported environment for fetch-token-intercept');
+  }
+}
+
 export function configure(config) {
   interceptor.configure(config);
 }
@@ -55,4 +58,8 @@ export function clear() {
   return interceptor.clear();
 }
 
+export {
+  isResponseUnauthorized,
+};
+
 init();

src/services/http.js

@@ -6,7 +6,7 @@ function isResponseStatus(response, status) {
     return false;
   }
 
-  return response['status'] === status;
+  return response.status === status;
 }
 
 export function isResponseOk(response) {