Skip to content

Commit 1299446

Browse files
committed
bug symfony#61062 [Brevo Mailer] Webhook IP Addresses have changed (richardhj)
This PR was merged into the 6.4 branch. Discussion ---------- [Brevo Mailer] Webhook IP Addresses have changed | Q | A | ------------- | --- | Branch? | 6.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Issues | | License | MIT So first I recognized multiple `RejectWebhookExceptions`. Then I checked my access logs and realized that webhook from Brevo can also come from the '172.246.240.1/20' IP range. This is also documented here: https://help.brevo.com/hc/en-us/articles/15127404548498-Brevo-IP-ranges-List-of-publicly-exposed-services This new IP range must have been added later this year, it hasn't been there in January, for instance: https://web.archive.org/web/20250125161029/https://help.brevo.com/hc/en-us/articles/15127404548498-Brevo-IP-ranges-List-of-publicly-exposed-services So this PR adds the new IP range for ingress webhook validation. <!-- 🛠️ Replace this text with a concise explanation of your change: - What it does and why it's needed - A simple example of how it works (include PHP, YAML, etc.) - If it modifies existing behavior, include a before/after comparison Contributor guidelines: - ✅ Add tests and ensure they pass - 🐞 Bug fixes must target the **lowest maintained** branch where they apply https://symfony.com/releases#maintained-symfony-branches - ✨ New features and deprecations must target the **feature** branch and must add an entry to the changelog file of the patched component: https://symfony.com/doc/current/contributing/code/conventions.html#writing-a-changelog-entry - 🔒 Do not break backward compatibility: https://symfony.com/bc --> Commits ------- 0cf7545 Update BrevoRequestParser.php
2 parents 7d1b50a + 0cf7545 commit 1299446

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/Symfony/Component/Mailer/Bridge/Brevo/Webhook/BrevoRequestParser.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ protected function getRequestMatcher(): RequestMatcherInterface
3737
new IsJsonRequestMatcher(),
3838
// https://developers.brevo.com/docs/how-to-use-webhooks#securing-your-webhooks
3939
// localhost is added for testing
40-
new IpsRequestMatcher(['185.107.232.1/24', '1.179.112.1/20', '127.0.0.1']),
40+
new IpsRequestMatcher(['185.107.232.1/24', '1.179.112.1/20', '172.246.240.1/20', '127.0.0.1']),
4141
]);
4242
}
4343

0 commit comments

Comments
 (0)