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

Create GitHub App Credentials Binding to support owner override

      Currently GitHub App credentials are being accessed through standard username and password credentials binding.

      It would be nice to create dedicated binding of it own to support owner override and other use cases in the future like overriding stale / expired settings. 

      We will know that we are done, where there is new binding created in github-branch-source-plugin repo and withCredentials can be used with the new binding. 

          [JENKINS-64662] Create GitHub App Credentials Binding to support owner override

          JENKINS-62220 is somewhat related to this story but I think the scope of 62220 is bigger than just creating the new binding, so I just created this one. 

          Naresh Rayapati added a comment - JENKINS-62220  is somewhat related to this story but I think the scope of 62220 is bigger than just creating the new binding, so I just created this one. 

          Naresh Rayapati added a comment - PR: https://github.com/jenkinsci/github-branch-source-plugin/pull/374  

          New PR created resolving conflicts after latest merges to master: 

          https://github.com/jenkinsci/github-branch-source-plugin/pull/375

          Naresh Rayapati added a comment - New PR created resolving conflicts after latest merges to master:  https://github.com/jenkinsci/github-branch-source-plugin/pull/375

          Dee Kryvenko added a comment -

          Hey nrayapati - my understanding this PR being stuck in review?...
          I just created a security vulnerability report here https://issues.jenkins.io/browse/JENKINS-66580 and it would be great if repository_ids can also be supplied to the new binding type.

          Dee Kryvenko added a comment - Hey nrayapati - my understanding this PR being stuck in review?... I just created a security vulnerability report here https://issues.jenkins.io/browse/JENKINS-66580 and it would be great if repository_ids can also be supplied to the new binding type.

          Jesse Glick added a comment -

          Since JENKINS-62220 was closed, is this still covering an important use case?

          Jesse Glick added a comment - Since JENKINS-62220 was closed, is this still covering an important use case?

            nrayapati Naresh Rayapati
            nrayapati Naresh Rayapati
            Votes:
            2 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated: