VDB
KO
MEDIUM

GHSA-64hg-93w9-fc35

Symfony's Mailjet Mailer Webhook Parser Never Verifies the Configured Secret — Unauthenticated Webhook Event Injection

Details

### Description

The Mailjet mailer bridge and the LOX24 SMS notifier bridge both ship webhook request parsers used to authenticate and decode the event callbacks each provider POSTs to an application's webhook endpoint. Their `doParse(Request $request, #[\SensitiveParameter] string $secret)` methods receive the configured webhook secret but never read it; they convert and return the payload unconditionally.

As a result, an application that wires up either webhook endpoint accepts **any** POST to that URL, even when a webhook secret is configured (the recommended setup). An attacker who knows the endpoint exists can submit forged event payloads, fake bounce / blocked / spam / open / click / delivery events, leading to suppression-list corruption, delivery-metrics fraud, etc.

### Resolution

`MailjetRequestParser::doParse()` now rejects the request unless it carries the expected HTTP Basic credentials, Mailjet's webhook authentication mechanism, using a constant-time comparison. The configured webhook secret is matched against the credentials embedded in the Mailjet webhook URL as `user:password` (use `:password` when the URL has no username).

`Lox24RequestParser::doParse()` now rejects the request unless it carries an `X-LOX24-Token` HTTP header whose value matches the configured secret, using a constant-time comparison. The same token must be configured in the LOX24 dashboard under the callback settings.

When no secret is configured the behaviour is unchanged: webhook authentication remains opt-in, but it is now actually enforced once opted in.

The Mailjet patch is available [here](https://github.com/symfony/symfony/commit/3e52bf5ab733ee32e35eeeeb2631d859c941838e) for branch 6.4.

The LOX24 patch is available [here](https://github.com/symfony/symfony/commit/4aaa45dd054f73445f1ab254968b7e60b546cc77) for branch 7.4 (the LOX24 bridge was introduced in 7.1 and is not present in 6.4).

### Credits

Symfony would like to thank Himanshu Anand for reporting the issue, and Alexandre Daubois and Nicolas Grekas for providing the fixes.

Are you affected?

Enter the version of the package you're using.

Affected packages

Packagist / symfony/lox24-notifier
Introduced in: 7.1.0 Fixed in: 7.4.12
Fix composer require symfony/lox24-notifier:^7.4.12
Packagist / symfony/lox24-notifier
Introduced in: 8.0.0 Fixed in: 8.0.12
Fix composer require symfony/lox24-notifier:^8.0.12
Packagist / symfony/symfony
Introduced in: 6.4.0 Fixed in: 6.4.40
Fix composer require symfony/symfony:^6.4.40
Packagist / symfony/symfony
Introduced in: 7.0.0 Fixed in: 7.4.12
Fix composer require symfony/symfony:^7.4.12
Packagist / symfony/symfony
Introduced in: 8.0.0 Fixed in: 8.0.12
Fix composer require symfony/symfony:^8.0.12
Packagist / symfony/mailjet-mailer
Introduced in: 6.4.0 Fixed in: 6.4.40
Fix composer require symfony/mailjet-mailer:^6.4.40
Packagist / symfony/mailjet-mailer
Introduced in: 7.0.0 Fixed in: 7.4.12
Fix composer require symfony/mailjet-mailer:^7.4.12
Packagist / symfony/mailjet-mailer
Introduced in: 8.0.0 Fixed in: 8.0.12
Fix composer require symfony/mailjet-mailer:^8.0.12

References