Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Add EventGrid compatible webhook format #1640

Merged
merged 5 commits into from
Feb 12, 2022

Conversation

stishkin
Copy link
Contributor

@stishkin stishkin commented Feb 5, 2022

Add an option to have webhooks posted in Azure Event Grid compatible format

Currently OneFuzz webhook data layout is not compatible with Azure Event Grid. Event Grid expects an array of objects and also the fields are named differently from Event Grid schema.

This a non-breaking change that adds an optional --message_format parameter to CLI to allow selection of existing OneFuzz webhook format or Event Grid friendly format. If not specified, then default is existing OneFuzz format.

This change makes webhooks more "Azure friendly".

@stishkin stishkin requested review from bmc-msft, ranweiler, chkeita and mgreisen and removed request for bmc-msft February 5, 2022 00:19
@mgreisen
Copy link
Contributor

mgreisen commented Feb 8, 2022

We should update the doc. https://github.com/microsoft/onefuzz/blob/main/docs/webhook_events.md

@stishkin stishkin force-pushed the event-grid-compatible-webhooks branch from 16f6634 to d3ae874 Compare February 11, 2022 22:39
@stishkin stishkin merged commit 77dcd57 into microsoft:main Feb 12, 2022
@stishkin stishkin deleted the event-grid-compatible-webhooks branch February 14, 2022 16:49
@ghost ghost locked as resolved and limited conversation to collaborators Mar 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants