Add validation logic for zoom-meeting-webhook-handler #19
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ebanner
force-pushed
the
zoom-meeting-webhook-handler-double-count-bug
branch
from
7b02900
to
8e91e73
Compare
Zoom requires Webhook validation for all webhooks created or modified after Oct 23, 2022. So in order for us to redeploy the zoom app, we will need this validation logic https://developers.zoom.us/docs/api/rest/webhook-reference/#validate-your-webhook-endpoint
danieltott
force-pushed
the
zoom-meeting-webhook-handler-double-count-bug
branch
from
8e91e73
to
b417ed9
Compare
danieltott
approved these changes
Apr 24, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds logic to respond to a zoom validation request with the appropriate response
Zoom requires Webhook validation for all webhooks created or modified after Oct 23, 2022 (documentation). The current webhook is presumably deployed before this date, so it's working fine without the validation logic (zoom will also send a revalidation request every 72 hours)
So in order for us to redeploy the zoom app, we would need to add this logic
In order for this to work, we need to add the
ZOOM_WEBHOOK_SECRET_TOKENto the.envor netlify environment variables. This token can be found on the zoom app's pageContext
This PR came about because we are trying to debug #15
We actually don't need to merge this PR in order to debug that issue. Only if we look at the prod netlify logs and they look truly head-scratching, then we can look into redeploying the zoom app to see if that magically fixes the issue