Skip to content
Permalink
Browse files

[FIXED JENKINS-41616] Non-trusted pull requests should use a probe ag…

…ainst the trusted revision not the PR's revision
  • Loading branch information...
stephenc committed Mar 15, 2017
1 parent 56aca14 commit 8272f20a872ce4fbafb6417b00bdd5aea0a43e7b
@@ -684,7 +684,7 @@ private void doRetrieve(SCMSourceCriteria criteria, SCMHeadObserver observer, Ta
// but this would be a lot more work, and is unlikely to differ from using refs/pull/123/merge:

Connector.checkApiRateLimit(listener, github);
try (SCMProbe probe = createProbe(head, null)) {
try (SCMProbe probe = createProbe(trusted ? head : head.getTarget(), null)) {
if (criteria.isHead(probe, listener)) {
// FYI https://developer.github.com/v3/pulls/#response-1
Boolean mergeable = ghPullRequest.getMergeable();

0 comments on commit 8272f20

Please sign in to comment.
You can’t perform that action at this time.