refactor(sdk)!: make resetAccessToken() sync

Change `MermaidChart#resetAccessToken()` so that it no longer returns a `Promise`.
Mermaid-Chart · Dec 4, 2023 · 203aabc · 203aabc
1 parent 8d6a1be
commit 203aabc
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/packages/sdk/CHANGELOG.md b/packages/sdk/CHANGELOG.md
@@ -5,6 +5,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Changes
+
+- `MermaidChart#resetAccessToken()` no longer returns a `Promise`.
+
 ### Added
 
 - Compile an ESM version of this codebase for Node.JS v18.

diff --git a/packages/sdk/src/index.ts b/packages/sdk/src/index.ts
@@ -57,8 +57,7 @@ export class MermaidChart {
     this.axios.interceptors.response.use((res: AxiosResponse) => {
       // Reset token if a 401 is thrown
       if (res.status === 401) {
-        // don't care if this function rejects/resolves
-        void this.resetAccessToken();
+        this.resetAccessToken();
       }
       return res;
     });
@@ -141,7 +140,7 @@ export class MermaidChart {
     this.accessToken = accessToken;
   }
 
-  public async resetAccessToken(): Promise<void> {
+  public resetAccessToken(): void {
     this.accessToken = undefined;
     this.axios.defaults.headers.common['Authorization'] = `Bearer none`;
   }