Merge pull request #68 from mosmohamed/header_credentials

Add credentials to fetch with cookies

.gitignore

@@ -1,3 +1,4 @@
 node_modules/
 dist/
 .cache/
+.idea/

CHANGELOG.md

@@ -6,6 +6,8 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased](https://github.com/pusher/push-notifications-web/compare/1.0.3...HEAD)
+ - Allow the fetch `credentials` option to be overidden in the default TokenProvider
+   implementation.
 
 ## [1.0.3](https://github.com/pusher/push-notifications-web/compare/1.0.2...1.0.3) - 2020-09-10
  - Fix bug in SDK where we weren't waiting for custom Service Workers to become

index.d.ts

@@ -9,6 +9,7 @@ interface TokenProviderOptions {
   url: string;
   queryParams?: { [key: string]: any };
   headers?: { [key: string]: string };
+  credentials?: string;
 }
 
 export class TokenProvider implements ITokenProvider {

src/do-request.js

@@ -1,7 +1,14 @@
-export default function doRequest({ method, path, body = null, headers = {} }) {
+export default function doRequest({
+  method,
+  path,
+  body = null,
+  headers = {},
+  credentials = 'same-origin',
+}) {
   const options = {
     method,
     headers,
+    credentials,
   };
 
   if (body !== null) {

src/token-provider.js

@@ -1,10 +1,11 @@
 import doRequest from './do-request';
 
 export default class TokenProvider {
-  constructor({ url, queryParams, headers } = {}) {
+  constructor({ url, queryParams, headers, credentials } = {}) {
     this.url = url;
     this.queryParams = queryParams;
     this.headers = headers;
+    this.credentials = credentials;
   }
 
   async fetchToken(userId) {
@@ -16,6 +17,7 @@ export default class TokenProvider {
       method: 'GET',
       path: `${this.url}?${encodedParams}`,
       headers: this.headers,
+      credentials: this.credentials,
     };
     let response = await doRequest(options);
     return response;