Merge pull request #416 from pavelvasev/fix_credentials

Bugfix: remove needless credentials argument
geotiffjs · Jan 19, 2024 · 4539582 · 4539582
2 parents b646d0b + 6855954
commit 4539582
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/source/client/base.js b/src/source/client/base.js
@@ -37,10 +37,10 @@ export class BaseClient {
 
   /**
    * Send a request with the options
-   * @param {{headers: HeadersInit, credentials: RequestCredentials, signal: AbortSignal}} [options={}]
+   * @param {{headers: HeadersInit, signal: AbortSignal}} [options={}]
    * @returns {Promise<BaseResponse>}
    */
-  async request({ headers, credentials, signal } = {}) { // eslint-disable-line no-unused-vars
+  async request({ headers, signal } = {}) { // eslint-disable-line no-unused-vars
     throw new Error('request is not implemented');
   }
 }
diff --git a/src/source/client/fetch.js b/src/source/client/fetch.js
@@ -33,12 +33,12 @@ export class FetchClient extends BaseClient {
   }
 
   /**
-   * @param {{headers: HeadersInit, credentials: RequestCredentials, signal: AbortSignal}} [options={}]
+   * @param {{headers: HeadersInit, signal: AbortSignal}} [options={}]
    * @returns {Promise<FetchResponse>}
    */
-  async request({ headers, credentials, signal } = {}) {
+  async request({ headers, signal } = {}) {
     const response = await fetch(this.url, {
-      headers, credentials, signal,
+      headers, credentials: this.credentials, signal,
     });
     return new FetchResponse(response);
   }