pagure

#1592 Include the pagure instance as well as the project in the notifications
Merged by pingou. Opened by pingou.
webhook_fixes  into  master

Download 1592.patch

These two information would allow one to have one endpoint receiving
notifications from multiple pagure instances.
The information is provided both in the headers as well as a the payload
(convenience and trust).

Fixes https://pagure.io/pagure/issue/1587

Perhaps call this X-Pagure-Instance ?

I went for this one to be consistent with the header we send in email notifications

Ah, fair enough.

+1

:joy:

\ó/

Thanks for the review :)

Pull-Request has been merged by pingou
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.