BCDevOps / bcdk

BC Developer Kit
Apache License 2.0
6 stars 15 forks source link

Jenkins BCDK default job incorrectly configuired discovering forked prs #22

Open patricksimonian opened 5 years ago

patricksimonian commented 5 years ago

in .jenkins/docker/contrib/configuration/jobs/_jobname/config.xml this line of code seems to allow any forked pr to kick of a build in the name space

  <org.jenkinsci.plugins.github__branch__source.ForkPullRequestDiscoveryTrait>
              <strategyId>2</strategyId>
              <trust class="org.jenkinsci.plugins.github_branch_source.ForkPullRequestDiscoveryTrait$TrustPermission"/>
   </org.jenkinsci.plugins.github__branch__source.ForkPullRequestDiscoveryTrait>

Why is this a problem?

Potential Fix

remove this config from the generator.

cvarjao commented 5 years ago

@patricksimonian , ForkPullRequestDiscoveryTrait$TrustPermission means only from "From users with Admin or Write permission". Did you observe anything different from that?