Skip to content

Tightens exception handling for Github Webhooks.#40

Merged
nukulb merged 1 commit intojenkinsci:masterfrom
hubba:master
May 1, 2014
Merged

Tightens exception handling for Github Webhooks.#40
nukulb merged 1 commit intojenkinsci:masterfrom
hubba:master

Conversation

@nukulb
Copy link
Member

@nukulb nukulb commented Mar 15, 2014

These are simply improvements so that other people have an easier time debugging their issues.

Tightens exception handling for Github Webhooks.

Checks to see if the Github Webhook request being processed is actually
of type 'push' before trying to process the request payload. The structure of
each type of Github Webhook differs and processing will fail.

Improves exception message when payload is not found and urges the
client to check the payload version on the Github Webhook to allow
easier debugging.

Checks to see if the Github Webhook request being processed is actually
of type 'push' before trying to process the request payload. The structure of
each type of Github Webhook differs and processing will fail.

Improves exception message when payload is not found and urges the
client to check the payload version on the Github Webhook to allow
easier debugging.
@cloudbees-pull-request-builder

plugins » github-plugin #32 SUCCESS
This pull request looks good

@jenkinsadmin
Copy link
Member

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

nukulb added a commit that referenced this pull request May 1, 2014
Tightens exception handling for Github Webhooks.
@nukulb nukulb merged commit 6553654 into jenkinsci:master May 1, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

Comments