Skip to content

feat(security/webhook/no-redir) Webhook should not follow redirections to prevent any risk of SSRF#343

Merged
leo-scalingo merged 1 commit into
masterfrom
feat/STORY-3848/webhook-no-redirect
May 28, 2026
Merged

feat(security/webhook/no-redir) Webhook should not follow redirections to prevent any risk of SSRF#343
leo-scalingo merged 1 commit into
masterfrom
feat/STORY-3848/webhook-no-redirect

Conversation

@leo-scalingo

Copy link
Copy Markdown
Member

No description provided.

@leo-scalingo leo-scalingo requested a review from a team as a code owner May 28, 2026 07:00
@leo-scalingo leo-scalingo requested review from SCedricThomas and removed request for a team May 28, 2026 07:00
@leo-scalingo leo-scalingo self-assigned this May 28, 2026
@notion-workspace

Copy link
Copy Markdown

curzolapierre
curzolapierre previously approved these changes May 28, 2026

@curzolapierre curzolapierre left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just a typo, otherwise LGTM

Comment thread plugin/webhook/webhook.go Outdated
@leo-scalingo leo-scalingo force-pushed the feat/STORY-3848/webhook-no-redirect branch from edf93fd to 44804b1 Compare May 28, 2026 15:16
@leo-scalingo leo-scalingo enabled auto-merge May 28, 2026 15:17
@leo-scalingo leo-scalingo merged commit 3a6f850 into master May 28, 2026
6 checks passed
@leo-scalingo leo-scalingo deleted the feat/STORY-3848/webhook-no-redirect branch May 28, 2026 15:50
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