Commit Graph

25 Commits

Author SHA1 Message Date
Nikita Popov e9f2676cb8 Revert "Debug webhook"
This reverts commit 9602517f18.
2021-03-30 21:58:07 +02:00
Nikita Popov 9602517f18 Debug webhook
Request goes through now, but is rejected by bugs.php.net.
2021-03-30 20:21:01 +02:00
Nikita Popov 1bbe009dfe Use file_get_contents instead of curl
Possibly not present on master?
2021-03-30 19:57:23 +02:00
Nikita Popov 8c7a80d162 Add webhook to auto-close bugs
This was previously handled as a post-receive hook. Now do this
in a push webhook instead.
2021-03-30 17:50:29 +02:00
Peter Kokot 0e511803a5 Refactor long array() syntax to short []
Since app is using PHP 5.4+ already, the longer `array()` syntax can be
refactored to shorter `[]`.
2018-12-10 03:26:03 +01:00
Nikita Popov d1bcd88aad Try to debug github webhook
Working fine when manually providing a payload, but I'm not getting
any mails...
2016-12-22 13:03:01 +01:00
Nikita Popov 3210ab0d11 Show "Issue" instead of "Pull Request" in a few more places 2016-12-22 00:17:46 +01:00
Nikita Popov 8724f96747 Retab file 2016-12-21 23:21:04 +01:00
Nikita Popov 6bd9ac552a Support issues in github webhook
It does not look like the webhook payloads explicitly tell you
whether an issue comment is on a issue or PR. As such, we use the
HTML URL to determine whether it's an issue or PR.
2016-12-21 23:18:05 +01:00
Nikita Popov 01d6ba78b9 Improve GitHub signature validation
We should not let the client decide the authentication method, see
https://www.wordfence.com/blog/2016/11/hacking-27-web-via-wordpress-auto-update/
2016-11-24 12:14:46 +01:00
Nikita Popov 8d52998596 Handle pull request comments as well
These are distinct from review comments.

I've cannibalized the base-branch in the mail subject to support
this easily. This does not seem like particularly important
information in most cases.
2016-11-15 21:52:34 +01:00
Nikita Popov 2320705c2b Try to make PR mails less spammy 2016-11-15 21:45:13 +01:00
Nikita Popov 4611f79bd1 Fix pull_request_review_comment handling 2016-11-15 21:07:20 +01:00
Hannes Magnusson aa6ec67f3b Removing the action (opened/closed/..) from the title
It otherwise creates tons of "gmail conversation"
2014-08-01 16:51:48 -07:00
kovacs.ferenc b1aecf8e19 duh, swap the lines so $payload is set before used 2014-08-01 16:25:01 +02:00
kovacs.ferenc 17134ef267 make sure that the variables used below are properly initialized 2014-08-01 15:27:52 +02:00
Hannes Magnusson f006fcfc79 Mailout the PR comments 2014-07-31 13:27:12 -07:00
kovacs.ferenc c5fe83adb0 send PRs of php-src to the git-pulls list 2014-06-11 20:49:09 +02:00
kovacs.ferenc cbc4e41a44 that was almost what I've planned 2014-06-11 00:30:36 +02:00
kovacs.ferenc 92e79aed06 it seems the -f is mandatory 2014-06-10 22:29:05 +02:00
kovacs.ferenc 5c92482bbe actually, I need two parts. :/ 2014-06-10 22:16:44 +02:00
kovacs.ferenc 5f50ef481f rename the variable holding the PR description to not collide with the request body variable with the same name 2014-06-10 22:00:03 +02:00
kovacs.ferenc 399ecb7fbc fix the signature verification 2014-06-10 20:50:50 +02:00
kovacs.ferenc acbafe1618 oops, thanks salathe 2014-06-10 20:08:16 +02:00
kovacs.ferenc 957a4a8351 add initial script, should be handle to multiple repositories and sending the PR notification mail to the matching address 2014-06-10 19:53:35 +02:00