Skip to content

Git-webhook-expand#342

Merged
orweis merged 6 commits into
masterfrom
git-webhook-expand
Dec 15, 2022
Merged

Git-webhook-expand#342
orweis merged 6 commits into
masterfrom
git-webhook-expand

Conversation

@orweis

@orweis orweis commented Dec 10, 2022

Copy link
Copy Markdown
Contributor

Add more webhook flows support
Signed-off-by: orweis orweis@gmail.com

Signed-off-by: orweis <orweis@gmail.com>
make the webhook flow toggled by GitWebhookRequestParams
Support both secret avlidationa s signature and as token

Signed-off-by: orweis <orweis@gmail.com>
@netlify

netlify Bot commented Dec 10, 2022

Copy link
Copy Markdown

Deploy Preview for opal-docs canceled.

Name Link
🔨 Latest commit 0787d71
🔍 Latest deploy log https://app.netlify.com/sites/opal-docs/deploys/639b272b8a45980009daabc1

@orweis
orweis requested a review from asafc December 10, 2022 22:47
Signed-off-by: orweis <orweis@gmail.com>
Signed-off-by: orweis <orweis@gmail.com>
Signed-off-by: orweis <orweis@gmail.com>

@asafc asafc left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM :)

Small nit comment

Comment thread packages/opal-server/opal_server/policy/webhook/api.py
@asafc
asafc marked this pull request as ready for review December 15, 2022 13:34
Signed-off-by: orweis <orweis@gmail.com>
@orweis
orweis merged commit 556f895 into master Dec 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants