fix: rename function to isValueUsedForBearerAuthentication, reverse logic

matt-the-dev · matt-the-dev · commit 42be5d6f958d · 2023-05-05T08:32:34.000-07:00
diff --git a/src/seam-connect/client.ts b/src/seam-connect/client.ts
@@ -148,10 +148,10 @@ export class Seam extends Routes {
   }
 }
 
-const isValueUsedForAPIKeyAuthentication = (apiKey: string) =>
-  !apiKey.startsWith("seam_at") &&
+const isValueUsedForBearerAuthentication = (value: string) =>
+  value.startsWith("seam_at") ||
   /** Exclude JWT tokens as well */
-  !apiKey.startsWith("ey")
+  value.startsWith("ey")
 
 const makeRequest = async <T>(
   client: Axios,
@@ -206,7 +206,7 @@ const getAuthHeaders = ({
       )
       return { "client-session-token": apiKey }
     }
-    if (isValueUsedForAPIKeyAuthentication(apiKey) && workspaceId)
+    if (!isValueUsedForBearerAuthentication(apiKey) && workspaceId)
       throw new Error(
         "You can't use API Key Authentication AND specify a workspace. Your API Key only works for the workspace it was created in. To use Session Key Authentication with multi-workspace support, contact Seam support."
       )

Original file line number	Diff line number	Diff line change
`@@ -148,10 +148,10 @@ export class Seam extends Routes {`
`148`	`148`	`}`
`149`	`149`	`}`
`150`	`150`
`151`		`-const isValueUsedForAPIKeyAuthentication = (apiKey: string) =>`
`152`		`- !apiKey.startsWith("seam_at") &&`
	`151`	`+const isValueUsedForBearerAuthentication = (value: string) =>`
	`152`	`+ value.startsWith("seam_at") \|\|`
`153`	`153`	`/** Exclude JWT tokens as well */`
`154`		`- !apiKey.startsWith("ey")`
	`154`	`+ value.startsWith("ey")`
`155`	`155`
`156`	`156`	`const makeRequest = async <T>(`
`157`	`157`	`client: Axios,`
`@@ -206,7 +206,7 @@ const getAuthHeaders = ({`
`206`	`206`	`)`
`207`	`207`	`return { "client-session-token": apiKey }`
`208`	`208`	`}`
`209`		`- if (isValueUsedForAPIKeyAuthentication(apiKey) && workspaceId)`
	`209`	`+ if (!isValueUsedForBearerAuthentication(apiKey) && workspaceId)`
`210`	`210`	`throw new Error(`
`211`	`211`	`"You can't use API Key Authentication AND specify a workspace. Your API Key only works for the workspace it was created in. To use Session Key Authentication with multi-workspace support, contact Seam support."`
`212`	`212`	`)`