[git-push webhooks] Add integration with readthedocs
Bug #1926431 reported by
Alvaro Uria
This bug affects 2 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Triaged
|
Low
|
Unassigned |
Bug Description
Git repositories can have a webhook configured. Optionally, a secret can be stored so the POST request can have a special header passed with a hash calculated from the mentioned secret value.
This approach does not work for generic webhooks APIs such as the readthedocs one [1].
When a push notification is received, launchpad should be able to run a similar command to:
curl -X POST -d 'branch=stable' -d "token=$TOKEN" https:/
As a wishlist, the webhook should be configurable for specific branches (e.g. the "stable" branch instead of any branch or the "master" one itself).
Changed in launchpad: | |
status: | New → Triaged |
importance: | Undecided → Low |
To post a comment you must log in.
I looked at this recently. I think a preferable option would be to add support to readthedocs to support Launchpad's webhook format, since it already has customized support for several other hosting sites, and this would allow better handling on their side anyway. https:/ /paste. ubuntu. com/p/wbJfBp9Gh 9/ is a start at this, but there are probably a few other bits I've missed, and it needs tests.