Anshul Patel
Anshul Patel
Upon further investigation, following changes worked for my setup ```diff diff --git a/modules/webhook/lambdas/webhook/src/webhook/handler.ts b/modules/webhook/lambdas/webhook/src/webhook/handler.ts index 983a11c..72d6cd4 100644 --- a/modules/webhook/lambdas/webhook/src/webhook/handler.ts +++ b/modules/webhook/lambdas/webhook/src/webhook/handler.ts @@ -96,7 +96,7 @@ async function verifySignature( body: string,...
Thanks @npalm for providing insights, Octokit itself says that sha1 is supported for legacy reasons reference: https://github.com/octokit/webhooks-methods.js/#sign
Sure @bjonnh-work , I will do this over the weekend
Hey @npalm, can you please help with the review: https://github.com/philips-labs/terraform-aws-github-runner/pull/2434? CC: @bjonnh-work