refactor(sdk)!: make resetAccessToken() sync

Change `MermaidChart#resetAccessToken()` so that it no longer returns a `Promise`.
Mermaid-Chart · Apr 22, 2024 · d536be5 · d536be5
1 parent 24055b2
commit d536be5
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`.
+
 ## [0.2.0] - 2024-04-11
 
 ### Added

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;
     });
@@ -153,7 +152,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`;
   }