From c8c25fd617d6737caa3c63835f9dbfa3161b5a8e Mon Sep 17 00:00:00 2001 From: Jesse Glick Date: Fri, 9 Jan 2026 15:25:24 -0500 Subject: [PATCH 1/2] Use `CredentialsProvider.findCredentialByIdInItemGroup` --- pom.xml | 11 +++++-- .../github/config/GitHubServerConfig.java | 31 ++++++------------- 2 files changed, 18 insertions(+), 24 deletions(-) diff --git a/pom.xml b/pom.xml index 9f40380f3..e19983797 100755 --- a/pom.xml +++ b/pom.xml @@ -52,7 +52,7 @@ jenkinsci/${project.artifactId}-plugin 2.504 - ${jenkins.baseline}.1 + ${jenkins.baseline}.3 false false v@{project.version} @@ -204,10 +204,17 @@ io.jenkins.tools.bom bom-${jenkins.baseline}.x - 4710.v016f0a_07e34d + 5804.v80587a_38d937 import pom + + + org.jenkins-ci.plugins + credentials + + 1469.vb_13b_4cfb_391d + diff --git a/src/main/java/org/jenkinsci/plugins/github/config/GitHubServerConfig.java b/src/main/java/org/jenkinsci/plugins/github/config/GitHubServerConfig.java index 9fed6de8d..9cb92a5d5 100644 --- a/src/main/java/org/jenkinsci/plugins/github/config/GitHubServerConfig.java +++ b/src/main/java/org/jenkinsci/plugins/github/config/GitHubServerConfig.java @@ -2,7 +2,6 @@ import com.cloudbees.plugins.credentials.CredentialsMatchers; import com.cloudbees.plugins.credentials.common.StandardListBoxModel; -import com.cloudbees.plugins.credentials.domains.DomainRequirement; import com.google.common.base.Function; import com.google.common.base.Optional; import com.google.common.base.Predicate; @@ -21,13 +20,10 @@ import java.io.IOException; import java.net.MalformedURLException; import java.net.URL; -import java.util.Collections; -import java.util.List; import jenkins.model.Jenkins; import jenkins.scm.api.SCMName; import org.apache.commons.lang3.StringUtils; import org.jenkinsci.plugins.github.internal.GitHubLoginFunction; -import org.jenkinsci.plugins.github.util.FluentIterableWrapper; import org.jenkinsci.plugins.github.util.misc.NullSafeFunction; import org.jenkinsci.plugins.github.util.misc.NullSafePredicate; import org.jenkinsci.plugins.plaincredentials.StringCredentials; @@ -41,14 +37,11 @@ import org.slf4j.Logger; import org.slf4j.LoggerFactory; -import static com.cloudbees.plugins.credentials.CredentialsMatchers.filter; -import static com.cloudbees.plugins.credentials.CredentialsMatchers.withId; -import static com.cloudbees.plugins.credentials.CredentialsProvider.lookupCredentials; +import static com.cloudbees.plugins.credentials.CredentialsProvider.findCredentialByIdInItemGroup; import static com.cloudbees.plugins.credentials.domains.URIRequirementBuilder.fromUri; import static org.apache.commons.lang3.StringUtils.defaultIfBlank; import static org.apache.commons.lang3.StringUtils.defaultIfEmpty; import static org.apache.commons.lang3.StringUtils.isNotBlank; -import static org.apache.commons.lang3.StringUtils.trimToEmpty; /** * This object represents configuration of each credentials-github pair. @@ -287,20 +280,14 @@ public Secret get() { */ @NonNull public static Optional secretFor(String credentialsId) { - List creds = filter( - lookupCredentials(StringCredentials.class, - Jenkins.getInstance(), ACL.SYSTEM, - Collections.emptyList()), - withId(trimToEmpty(credentialsId)) - ); - - return FluentIterableWrapper.from(creds) - .transform(new NullSafeFunction() { - @Override - protected Secret applyNullSafe(@NonNull StringCredentials input) { - return input.getSecret(); - } - }).first(); + if (credentialsId == null) { + return Optional.absent(); + } + var creds = findCredentialByIdInItemGroup(credentialsId, StringCredentials.class, null, null, null); + if (creds == null) { + return Optional.absent(); + } + return Optional.of(creds.getSecret()); } /** From 06b0574d66b6514103448d3633461583708be313 Mon Sep 17 00:00:00 2001 From: Jesse Glick Date: Mon, 12 Jan 2026 12:54:38 -0500 Subject: [PATCH 2/2] Dep released --- pom.xml | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/pom.xml b/pom.xml index e19983797..edb5325a8 100755 --- a/pom.xml +++ b/pom.xml @@ -212,8 +212,7 @@ org.jenkins-ci.plugins credentials - - 1469.vb_13b_4cfb_391d + 1480.v2246fd131e83