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

Re-enable GitHub PR support for public repos

    XMLWordPrintable

    Details

    • Similar Issues:

      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

            Activity

            Hide
            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

            Show
            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
            Hide
            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.

            Show
            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.
            Hide
            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

            Show
            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
            Hide
            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

            Show
            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
            Hide
            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).

            Show
            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 ).

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: