-
-
Notifications
You must be signed in to change notification settings - Fork 6.1k
Description
The body is empty in the webhook payload for pull request merged / closed events.
This is causing a problem with the redmine_merge_request_links plugin: tf/redmine_merge_request_links#27
At first I thought that this behavior maybe was intentional, but looking at the code, I don't think it is:
gitea/modules/notification/mail/mail.go
Line 127 in c61b460
pr.Issue.Content = "" |
The pr.Issue.Content is emptied in the e-mail notification code, and as a side-effect the issue content is also empty for notifiers that are called afterwards (indexer, webhook and action).
Such unexpected manipulations of the notification input also happens here:
gitea/modules/notification/mail/mail.go
Line 57 in c61b460
issue.Content = "" |
gitea/modules/notification/mail/mail.go
Line 154 in c61b460
comment.Content = "" |