Repository navigation
docs(github-app): webhook correction - #301
Merged
Merged
Conversation
Update GitHub App documentation to reflect that a GitHub App already has a built-in webhook that does not require creating a new webhook using the _Webhook_ permission. Furthermore, to take advantage of that built-in webhook, events must be enabled on the App's _Permissions_ page. Otherwise, if you wanted to use a separate webhook created by Jenkins using the credentials of the GitHub App, you would need to enable the _Admin_ permission, or else you would get the following error and no webhook: ``` Failed to add GitHub webhook for GitHubRepositoryName[host=github.example.com,username=test,repository=test] java.lang.NullPointerException: There are no credentials with admin access to manage hooks on GitHubRepositoryName[host=github.example.com,username=test,repository=test] at com.google.common.base.Preconditions.checkNotNull(Preconditions.java:231) at org.jenkinsci.plugins.github.webhook.WebhookManager$2.applyNullSafe(WebhookManager.java:179) at org.jenkinsci.plugins.github.webhook.WebhookManager$2.applyNullSafe(WebhookManager.java:175) ```
hutson
marked this pull request as ready for review
May 29, 2020 06:12
bitwiseman
approved these changes
May 29, 2020
|
I am seeing still the error though. It seems like the plugin is trying to register a webhook even though it is not needed with github app? |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Update GitHub App documentation to reflect that a GitHub App already has a built-in webhook that does not require creating a new webhook using the Webhook permission. Furthermore, to take advantage of that built-in webhook, events must be enabled on the App's Permissions page.
Otherwise, if you wanted to use a separate webhook created by Jenkins using the credentials of the GitHub App, you would need to enable the Admin permission, or else you would get the following error and no webhook:
Description
A brief summary describing the changes in this pull request. See
JENKINS-XXXXX for further information.
Submitter checklist
Reviewer checklist
Documentation changes
Users/aliases to notify