Bugfix for newly hosted webhook handler. (#68)

Author: themarolt

backend/src/api/apiResponseHandler.ts

@@ -15,6 +15,8 @@ export default class ApiResponseHandler {
   }
 
   static async error(_req, res, error) {
+    console.log('API ERROR', error)
+
     if (error && [400, 401, 403, 404].includes(error.code)) {
       res.status(error.code).send(error.message)
     } else {

backend/src/serverless/integrations/webhooks/github.ts

@@ -491,8 +491,10 @@ export default class GitHubWebhook {
     try {
       const signature = req.headers['x-hub-signature']
       const secret = GITHUB_CONFIG.webhookSecret
+
       console.log('Verifying webhook...')
-      const isVerified = verifyGithubWebhook(signature, req.body, secret) // Returns true if verification succeeds; otherwise, false.
+      const isVerified = verifyGithubWebhook(signature, JSON.stringify(req.body), secret) // Returns true if verification succeeds; otherwise, false.
+
       console.log('Verification', isVerified)
       if (!isVerified) {
         throw new Error('Webhook not verified')

backend/src/serverless/integrations/workers/githubWebhookWorker.ts

@@ -2,6 +2,6 @@ import GitHubWebhook from '../webhooks/github'
 
 export default async function githubWebhookWorker(req) {
   GitHubWebhook.verify(req)
-  const result = await new GitHubWebhook(req.headers['x-github-event'], JSON.parse(req.body)).main()
+  const result = await new GitHubWebhook(req.headers['x-github-event'], req.body).main()
   return result
 }