Skip to content

Commit d987754

Browse files
authored
fix: Verify signature of payloads containing control characters (#39)
1 parent 9865ef7 commit d987754

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

index.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ class ScmRouter extends Scm {
9191
* choose webhook scm module
9292
* @method chooseWebhookScm
9393
* @param {Object} headers The request headers associated with the webhook payload
94-
* @param {Object} payload The webhook payload received from the SCM service
94+
* @param {String} payload The webhook payload received from the SCM service
9595
* @return {Promise} scm object
9696
*/
9797
chooseWebhookScm(headers, payload) {
@@ -205,7 +205,7 @@ class ScmRouter extends Scm {
205205
* Parse the webhook for the specific source control
206206
* @method _parseHook
207207
* @param {Object} headers The request headers associated with the webhook payload
208-
* @param {Object} payload The webhook payload received from the SCM service
208+
* @param {String} payload The webhook payload received from the SCM service
209209
* @return {Promise}
210210
*/
211211
_parseHook(headers, payload) {
@@ -428,7 +428,7 @@ class ScmRouter extends Scm {
428428
* Determine a scm module can handle the received webhook
429429
* @method _canHandleWebhook
430430
* @param {Object} headers The request headers associated with the webhook payload
431-
* @param {Object} payload The webhook payload received from the SCM service
431+
* @param {String} payload The webhook payload received from the SCM service
432432
* @return {Promise}
433433
*/
434434
_canHandleWebhook(headers, payload) {

0 commit comments

Comments
 (0)