Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-46364

GitHub Branch Source Plugin can't create status if credential restricted by spec

      There appears to be an issue with the github-branch-source plugin where if you put a specification on a credential's domain, the credential will never be selected to authenticate when creating a GitHub Status. This appears to be because the apiUrl is not being passed when looking up the credential.

      The project build log will display:

      Could not update commit status. Message: {"message":"Must authenticate to access this API.","documentation_url":"https://developer.github.com/enterprise/2.9/v3"}
      

      I've got a patch incoming and have tested it out for a hostname specification and was able to successfully have it fail before and succeed after the patch. I also reverted to 2.2.2 and the failure returned.

          [JENKINS-46364] GitHub Branch Source Plugin can't create status if credential restricted by spec

          I have created a pull request to resolve this issue.

          Justin Harringa added a comment - I have created a pull request to resolve this issue.

          Code changed in jenkins
          User: Stephen Connolly
          Path:
          src/main/java/org/jenkinsci/plugins/github_branch_source/GitHubBuildStatusNotification.java
          http://jenkins-ci.org/commit/github-branch-source-plugin/68103188034a0fac323ce2111ae462c3c8116cd8
          Log:
          Merge pull request #155 from justinharringa/JENKINS-46364

          JENKINS-46364 Connector.lookupScanCredentials - pass apiUrl()

          Compare: https://github.com/jenkinsci/github-branch-source-plugin/compare/ced0cb0dd661...68103188034a

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Stephen Connolly Path: src/main/java/org/jenkinsci/plugins/github_branch_source/GitHubBuildStatusNotification.java http://jenkins-ci.org/commit/github-branch-source-plugin/68103188034a0fac323ce2111ae462c3c8116cd8 Log: Merge pull request #155 from justinharringa/ JENKINS-46364 JENKINS-46364 Connector.lookupScanCredentials - pass apiUrl() Compare: https://github.com/jenkinsci/github-branch-source-plugin/compare/ced0cb0dd661...68103188034a

            Unassigned Unassigned
            justinharringa Justin Harringa
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: