From d16e72b39535d0ec9165e0e6db1bd2911c90ef23 Mon Sep 17 00:00:00 2001
From: ivinokur <ivinokur@redhat.com>
Date: Wed, 20 Mar 2024 12:13:28 +0200
Subject: [PATCH] fixup! Add provider name annotation to PAT secret

---
 .../github/AbstractGithubPersonalAccessTokenFetcher.java       | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/wsmaster/che-core-api-factory-github-common/src/main/java/org/eclipse/che/api/factory/server/github/AbstractGithubPersonalAccessTokenFetcher.java b/wsmaster/che-core-api-factory-github-common/src/main/java/org/eclipse/che/api/factory/server/github/AbstractGithubPersonalAccessTokenFetcher.java
index 05b6dd0c49..d8873549fd 100644
--- a/wsmaster/che-core-api-factory-github-common/src/main/java/org/eclipse/che/api/factory/server/github/AbstractGithubPersonalAccessTokenFetcher.java
+++ b/wsmaster/che-core-api-factory-github-common/src/main/java/org/eclipse/che/api/factory/server/github/AbstractGithubPersonalAccessTokenFetcher.java
@@ -142,6 +142,9 @@ public abstract class AbstractGithubPersonalAccessTokenFetcher
           isValid(
               new PersonalAccessTokenParams(
                   scmServerUrl,
+                  // Despite the fact that we may have two GitHub oauth providers, we always set
+                  // "github" to the token provider name. The specific GitHub oauth provider
+                  // references to the specific token by the url parameter.
                   OAUTH_PROVIDER_NAME,
                   tokenName,
                   tokenId,