Nikita Popov
369ff2016a
Try adding Reply-To header to commit mails
...
Not sure whether this will get stripped by the mailing list, but
let's try it. I don't want to specify the author/committer email
in From, as it'll likely run afoul of DMARC.
2021-04-02 22:43:57 +02:00
Nikita Popov
89fd8c2964
Try to fix mail charset issue
...
Set utf-8 as the charset when using PHPMailer. The manual encoding
code was a leftover from direct calls to mail() and only handles
the charset in the subject but not other headers.
2021-04-02 22:08:41 +02:00
Nikita Popov
d0cac5411f
Make "Fixed bug" check case-insensitive
2021-04-01 22:26:49 +02:00
Nikita Popov
6488d65ae6
Handle optional github user name
...
Author/committer may not necessarily have a github account.
2021-04-01 00:26:34 +02:00
Nikita Popov
e22ac4f3a5
Also include pusher name
2021-03-31 21:03:17 +02:00
Nikita Popov
c0b0e18a4c
Disable diffs, enable general commit mails
...
Fetching diffs fails on master.php.net -- I suspect that this is
because it's too old to support TLS 1.2. Inline diff display
should be enabled once the box has been migrated.
2021-03-31 18:29:07 +02:00
Nikita Popov
2cab6e865a
Send commit mail
2021-03-31 17:13:25 +02:00
Nikita Popov
cfbbcff722
Send mail for ref create/delete/force
2021-03-31 15:33:09 +02:00
Nikita Popov
340756d5dd
Remove bogus headers argument
2021-03-30 23:59:03 +02:00
Nikita Popov
e887adfd44
Switch webhook to PHPMailer
2021-03-30 22:04:03 +02:00
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
