Skip to content

Support GitHub apps with multiple organization installations#75

Merged
bluesliverx merged 3 commits intojenkinsci:masterfrom
bluesliverx:master
Mar 6, 2025
Merged

Support GitHub apps with multiple organization installations#75
bluesliverx merged 3 commits intojenkinsci:masterfrom
bluesliverx:master

Conversation

@bluesliverx
Copy link
Contributor

@bluesliverx bluesliverx commented Mar 6, 2025

A deprecated method was being used which prevented usage of a GitHub app that is installed in multiple orgs but the credential does not specify an owner (i.e. configured to infer the org to use from the repo). This uses the new method so that the inference can be done correctly.

Testing done

I installed this manually on a Jenkins server that is exhibiting the problem and then ensured that it worked with the change.

Submitter checklist

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests - that demonstrates feature works or fixes the issue

@bluesliverx bluesliverx requested a review from a team as a code owner March 6, 2025 20:33
@bluesliverx bluesliverx merged commit 63854e2 into jenkinsci:master Mar 6, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant