abiosoft / abiosoft/caddy-git

Travis webhook verification

Aperta
#87 1 commento 0 reazioni 0 assegnatari Vedi su GitHub
bug
Lingua principale
Go
Stelle
206
Fork
42
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

https://github.com/abiosoft/caddy-git/blob/7c37f04cfac86475f954286aa1b7e7aba0fa1ca9/travis_hook.go#L81

Is the `handleSignature` function correct? The [Travis documentation](https://docs.travis-ci.com/user/notifications/#Verifying-Webhook-requests) seems to imply the code should check the `Signature` header against Travis’ public key.

It seems like the `Authorization` header is an [old method of authenticating Travis requests](https://blog.travis-ci.com/2016-08-31-webhook-delivery-changes) phased out in November 2016.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.