Skip to content

Commit c83c774

Browse files
committed
feat(middleware): emit "error" event on verification error
1 parent 4870c65 commit c83c774

File tree

1 file changed

+4
-14
lines changed

1 file changed

+4
-14
lines changed

middleware/middleware.js

Lines changed: 4 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ module.exports = middleware
22

33
const isntWebhook = require('./isnt-webhook')
44
const getMissingHeaders = require('./get-missing-headers')
5-
const verify = require('../verify')
5+
const verifyAndReceive = require('./verify-and-receive')
66

77
const debug = require('debug')('webhooks:receiver')
88
function middleware (state, request, response, next) {
@@ -49,22 +49,12 @@ function middleware (state, request, response, next) {
4949

5050
request.on('end', () => {
5151
const payload = Buffer.concat(dataChunks).toString()
52-
const matchesSignature = verify(
53-
state.secret,
54-
payload,
55-
signature
56-
)
5752

58-
if (!matchesSignature) {
59-
response.statusCode = 400
60-
response.end('x-hub-signature does not match event payload and secret')
61-
return
62-
}
63-
64-
state.eventHandler.receive({
53+
verifyAndReceive(state, {
6554
id: id,
6655
name: eventName,
67-
payload: JSON.parse(payload)
56+
payload: JSON.parse(payload),
57+
signature
6858
})
6959

7060
.then(() => {

0 commit comments

Comments
 (0)