Skip to content

Commit

Permalink
Revert "Do not enforce GitHub app to comes from the same org as the r…
Browse files Browse the repository at this point in the history
…epo org. The GH app can come from another org as log as it is installed in the org with the target git repo there is no security issue (#744)"

This reverts commit 4c25095.
  • Loading branch information
jtnord authored Nov 15, 2023
1 parent 6b61a1d commit 901162c
Showing 1 changed file with 3 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -326,6 +326,9 @@ public String getUsername() {
@NonNull
public synchronized GitHubAppCredentials withOwner(@NonNull String owner) {
if (this.owner != null) {
if (!owner.equals(this.owner)) {
throw new IllegalArgumentException("Owner mismatch: " + this.owner + " vs. " + owner);

Check warning on line 330 in src/main/java/org/jenkinsci/plugins/github_branch_source/GitHubAppCredentials.java

View check run for this annotation

ci.jenkins.io / Code Coverage

Not covered lines

Lines 329-330 are not covered by tests
}
return this;
}
if (byOwner == null) {
Expand Down

0 comments on commit 901162c

Please sign in to comment.