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

Re-enable GitHub PR support for public repos

    XMLWordPrintable

Details

    Description

      GitHub PRs for public repos were disabled in the 1.3 release due to concerns about untrusted Jenkinsfiles. While that's understandable, the result is that, well, this doesn't build PRs any more unless you're using a private repo, and that's unfortunate to say the least. This is especially inconvenient for a GitHub Enterprise use case, since "public repos" doesn't mean the same thing there.

      Attachments

        Issue Links

          depends on

          Improvement - An improvement or enhancement to an existing feature or task. JENKINS-34596 Set visible flag when Jenkinsfile is taken from a trusted revision rather than PR head

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved
          links to

          Web Link github-branch-source PR 29

          Web Link scm-api-plugin PR 5

          Web Link workflow-plugin PR 244

          Activity

            Descending order - Click to sort in ascending order
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Jesse Glick
            Path:
            demo/README.md
            http://jenkins-ci.org/commit/github-branch-source-plugin/4970dbc528f0d26ffe0f6ebaac8ac9959c14f04c
            Log:
            Removing instructions obsolete since #29 (JENKINS-33256).

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: demo/README.md http://jenkins-ci.org/commit/github-branch-source-plugin/4970dbc528f0d26ffe0f6ebaac8ac9959c14f04c Log: Removing instructions obsolete since #29 ( JENKINS-33256 ).
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Jesse Glick
            Path:
            multibranch/src/main/java/org/jenkinsci/plugins/workflow/multibranch/SCMBinder.java
            multibranch/src/test/java/org/jenkinsci/plugins/workflow/multibranch/SCMBinderTest.java
            http://jenkins-ci.org/commit/workflow-multibranch-plugin/5d458888fe0eba9e1ed2306328beb914674ede64
            Log:
            Merge pull request #244 from jglick/getTrustedRevision

            JENKINS-33256 Calling SCMSource.getTrustedRevision
            Originally-Committed-As: 76bdc6f044f2eaf60f0f313cb09c7e006802e9a5

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: multibranch/src/main/java/org/jenkinsci/plugins/workflow/multibranch/SCMBinder.java multibranch/src/test/java/org/jenkinsci/plugins/workflow/multibranch/SCMBinderTest.java http://jenkins-ci.org/commit/workflow-multibranch-plugin/5d458888fe0eba9e1ed2306328beb914674ede64 Log: Merge pull request #244 from jglick/getTrustedRevision JENKINS-33256 Calling SCMSource.getTrustedRevision Originally-Committed-As: 76bdc6f044f2eaf60f0f313cb09c7e006802e9a5
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Jesse Glick
            Path:
            pom.xml
            src/main/java/org/jenkinsci/plugins/github_branch_source/GitHubSCMSource.java
            src/main/java/org/jenkinsci/plugins/github_branch_source/UntrustedPullRequestSCMRevision.java
            http://jenkins-ci.org/commit/github-branch-source-plugin/e1ea086b75541f686c21d8604a454f33b933dadc
            Log:
            Merge pull request #29 from jglick/getTrustedRevision-JENKINS-33256

            JENKINS-33256 Untrusted PRs

            Compare: https://github.com/jenkinsci/github-branch-source-plugin/compare/3df443cd38c6...e1ea086b7554

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: pom.xml src/main/java/org/jenkinsci/plugins/github_branch_source/GitHubSCMSource.java src/main/java/org/jenkinsci/plugins/github_branch_source/UntrustedPullRequestSCMRevision.java http://jenkins-ci.org/commit/github-branch-source-plugin/e1ea086b75541f686c21d8604a454f33b933dadc Log: Merge pull request #29 from jglick/getTrustedRevision- JENKINS-33256 JENKINS-33256 Untrusted PRs Compare: https://github.com/jenkinsci/github-branch-source-plugin/compare/3df443cd38c6...e1ea086b7554
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Jesse Glick
            Path:
            src/main/java/org/jenkinsci/plugins/github_branch_source/GitHubSCMSource.java
            http://jenkins-ci.org/commit/github-branch-source-plugin/3e23edf68e0c87b857c4923dabe998167cc26150
            Log:
            [FIXED JENKINS-33256] Reënabling PRs on public repos.

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: src/main/java/org/jenkinsci/plugins/github_branch_source/GitHubSCMSource.java http://jenkins-ci.org/commit/github-branch-source-plugin/3e23edf68e0c87b857c4923dabe998167cc26150 Log: [FIXED JENKINS-33256] Reënabling PRs on public repos.
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Jesse Glick
            Path:
            CHANGES.md
            multibranch/src/main/java/org/jenkinsci/plugins/workflow/multibranch/SCMBinder.java
            multibranch/src/test/java/org/jenkinsci/plugins/workflow/multibranch/SCMBinderTest.java
            pom.xml
            http://jenkins-ci.org/commit/workflow-plugin/76bdc6f044f2eaf60f0f313cb09c7e006802e9a5
            Log:
            Merge pull request #244 from jglick/getTrustedRevision

            JENKINS-33256 Calling SCMSource.getTrustedRevision

            Compare: https://github.com/jenkinsci/workflow-plugin/compare/f19e83d255e2...76bdc6f044f2

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: CHANGES.md multibranch/src/main/java/org/jenkinsci/plugins/workflow/multibranch/SCMBinder.java multibranch/src/test/java/org/jenkinsci/plugins/workflow/multibranch/SCMBinderTest.java pom.xml http://jenkins-ci.org/commit/workflow-plugin/76bdc6f044f2eaf60f0f313cb09c7e006802e9a5 Log: Merge pull request #244 from jglick/getTrustedRevision JENKINS-33256 Calling SCMSource.getTrustedRevision Compare: https://github.com/jenkinsci/workflow-plugin/compare/f19e83d255e2...76bdc6f044f2
            View 6 older comments

            People

              jglick Jesse Glick
              abayer Andrew Bayer
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: