Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Zoom web-hook Sentinel connector does not pass Zoom validation #9163

Closed
aka0 opened this issue Oct 5, 2023 · 14 comments
Closed

Zoom web-hook Sentinel connector does not pass Zoom validation #9163

aka0 opened this issue Oct 5, 2023 · 14 comments
Assignees
Labels
Connector Connector specialty review needed

Comments

@aka0
Copy link

aka0 commented Oct 5, 2023

Describe the bug
On Oct 23, 2022, Zoom requires web-hook destination in order receive web-hook events. As such, newly deployed Zoom connector would not be able to receive events.

To Reproduce
Steps to reproduce the behavior:

  1. Select ‘Develop’ in the top right-hand corner and click ‘Build App’.
  2. Select ‘Webhook Only’ as your app type.
  3. Give your app a name.
  4. Fill out the required Basic Information and click continue.
  5. Under the Feature Tab enable the ‘Event Subscriptions’ toggle and click ‘Add new event subscription’.
  6. Set a subscription name and in the Event notification endpoint URL enter your Function App URL.
  7. Click Validate

Validation will fail

Expected behavior
Validation passes

Screenshots
If applicable, add screenshots to help explain your problem.

image

Additional context

  1. https://devsupport.zoom.us/hc/en-us/articles/360060332591-How-to-add-Webhooks-to-an-app
  2. https://developers.zoom.us/docs/api/rest/webhook-reference/#validate-your-webhook-endpoint
@github-actions
Copy link
Contributor

github-actions bot commented Oct 5, 2023

Thank you for submitting an Issue to the Azure Sentinel GitHub repo! You should expect an initial response to your Issue from the team within 5 business days. Note that this response may be delayed during holiday periods. For urgent, production-affecting issues please raise a support ticket via the Azure Portal.

@v-sudkharat
Copy link
Contributor

Hi @aka0, thanks for flagging this issue, we will soon get back to you on this. Thanks!

@v-sudkharat v-sudkharat added the Connector Connector specialty review needed label Oct 6, 2023
@v-muuppugund
Copy link
Contributor

Hi @aka0 ,Could you please confirm whether we are using zoom reports solution in sentnel?

@aka0
Copy link
Author

aka0 commented Oct 6, 2023

Hi @aka0 ,Could you please confirm whether we are using zoom reports solution in sentnel?

No. I was trying to use Zoom data connector (web-hook).

See: https://github.com/Azure/Azure-Sentinel/blob/master/DataConnectors/Zoom/ZoomLogs/run.ps1

ZoomReport solution also does not work because Zoom has deprecated JWT authentication in June 2023.

Ref: https://developers.zoom.us/blog/jwt-deprecation-guide/

@v-muuppugund
Copy link
Contributor

HI

Hi @aka0 ,Could you please confirm whether we are using zoom reports solution in sentnel?

No. I was trying to use Zoom data connector (web-hook).

See: https://github.com/Azure/Azure-Sentinel/blob/master/DataConnectors/Zoom/ZoomLogs/run.ps1

ZoomReport solution also does not work because Zoom has deprecated JWT authentication in June 2023.

Ref: https://developers.zoom.us/blog/jwt-deprecation-guide/

@aka0 Working on the https://github.com/Azure/Azure-Sentinel/blob/master/DataConnectors/Zoom/ZoomLogs/run.ps1 issue, will update you

ZoomReport solution also does not work because Zoom has deprecated JWT authentication in June 2023. its development done and we are working to production it, soon you will get the production version.

@v-muuppugund
Copy link
Contributor

HI

Hi @aka0 ,Could you please confirm whether we are using zoom reports solution in sentnel?

No. I was trying to use Zoom data connector (web-hook).
See: https://github.com/Azure/Azure-Sentinel/blob/master/DataConnectors/Zoom/ZoomLogs/run.ps1
ZoomReport solution also does not work because Zoom has deprecated JWT authentication in June 2023.
Ref: https://developers.zoom.us/blog/jwt-deprecation-guide/

@aka0 Working on the https://github.com/Azure/Azure-Sentinel/blob/master/DataConnectors/Zoom/ZoomLogs/run.ps1 issue, will update you

ZoomReport solution also does not work because Zoom has deprecated JWT authentication in June 2023. its development done and we are working to production it, soon you will get the production version.

Hi @aka0 ,still working on the validation issue at zoom side and next update is 13/10/2023

@aka0
Copy link
Author

aka0 commented Oct 12, 2023

Hi @v-muuppugund ,

I added validation to the Zoom webhook connector. See PR: #9201

@v-muuppugund
Copy link
Contributor

Hi @v-muuppugund ,

I added validation to the Zoom webhook connector. See PR: #9201

Hi @aka0 , ok, just want to update, actually gone through the changes/code of zoom logs at https://developers.zoom.us/docs/api/rest/webhook-reference/#validate-your-webhook-endpoint ,thought of changing the code in couple of days ,as working on other priority items,Thanks

@v-muuppugund
Copy link
Contributor

Hi @aka0 ,This PR has been reviewed,Could you please check the review comments and close it,Thanks

@v-muuppugund
Copy link
Contributor

Hi @aka0 ,got update reg PR comments resolution and team is working on testing it and may require some testing screen shots.

@v-muuppugund
Copy link
Contributor

Hi @aka0 ,got update reg PR comments resolution and team is working on testing it and may require some testing screen shots.

Hi @aka0 ,reviewed the PR again,please resolve the comments and let us share convenient time for meeting for the same for any queries

@v-muuppugund
Copy link
Contributor

v-muuppugund commented Nov 3, 2023

Hi @aka0 ,As discussed in today's call reviewed the PR and resolved few comments and will be working on the below changes
1.New version package
2.Short link creation
3.Pointing package with new target
4.Arm template updates
5.Schedule call with you on monday i.e 06Nov23 for testing it e2e this pr

@v-muuppugund
Copy link
Contributor

v-muuppugund commented Nov 6, 2023

Hi @aka0 ,As discussed,scheduled call for this issue on tuesday i.e. 11/7/23

@v-muuppugund
Copy link
Contributor

Made changes in PR, reviewed,tested with author,so closing the issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Connector Connector specialty review needed
Projects
None yet
Development

No branches or pull requests

4 participants