Merge pull request #74 from daniel-beck/notify-commit-external-creden…

…tials [FIXED JENKINS-21785] Check for changes in folders linked via svn:externals fails due to missing credentials - Typo in original code and externals credentials were not being resolved correctly. Good catch by @daniel-beck
jenkinsci · Apr 7, 2014 · a923c50 · a923c50 · daniel-beck · Apr 7, 2014
2 parents 0a17b93 + 69ef77b
commit a923c50
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/main/java/hudson/scm/SubversionSCM.java b/src/main/java/hudson/scm/SubversionSCM.java
@@ -2736,7 +2736,7 @@ public SVNRepository openRepository(AbstractProject context, boolean keepConnect
                         StandardCredentials cred = CredentialsMatchers
                                 .firstOrNull(CredentialsProvider.lookupCredentials(StandardCredentials.class, context,
                                         ACL.SYSTEM, Collections.<DomainRequirement>emptyList()),
-                                        CredentialsMatchers.allOf(CredentialsMatchers.withId(credentialsId),
+                                        CredentialsMatchers.allOf(CredentialsMatchers.withId(c.getCredentialsId()),
                                                 CredentialsMatchers.anyOf(CredentialsMatchers.instanceOf(
                                                         StandardCredentials.class), CredentialsMatchers.instanceOf(
                                                         SSHUserPrivateKey.class))));