Dedupe saml error check (#6551)

microsoft · Dec 16, 2024 · 798da62 · 798da62
1 parent 7c540d3
commit 798da62
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/common/authentication.ts b/src/common/authentication.ts
@@ -23,5 +23,5 @@ export class AuthenticationError extends Error {
 }
 
 export function isSamlError(e: { message?: string }): boolean {
-	return !!e.message?.startsWith('Resource protected by organization SAML enforcement.');
+	return !!e.message?.includes('Resource protected by organization SAML enforcement.');
 }
diff --git a/src/github/githubRepository.ts b/src/github/githubRepository.ts
@@ -1493,7 +1493,7 @@ export class GitHubRepository extends Disposable {
 				},
 			}, true); // There's an issue with the GetChecks that can result in SAML errors.
 		} catch (e) {
-			if (e.message?.startsWith('GraphQL error: Resource protected by organization SAML enforcement.')) {
+			if (isSamlError(e.message)) {
 				// There seems to be an issue with fetching status checks if you haven't SAML'd with every org you have
 				// The issue is specifically with the CheckSuite property. Make the query again, but without that property.
 				if (!captureUseFallbackChecks) {