bug symfony#54089 [Mailer] [Brevo] Remove tags from mandatory event arguments (palgalik)

fabpot · fabpot · commit 182e93e3e5e1 · 2024-02-28T07:58:57.000+01:00
This PR was merged into the 6.4 branch. Discussion ---------- [Mailer] [Brevo] Remove tags from mandatory event arguments 'tags' in not mandatory part of an event therefore it got removed from the validation process | Q | A | ------------- | --- | Branch? | 6.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Issues | Fix [54088](symfony#54088) | License | MIT Commits ------- dfb41cd [Mailer][Brevo] Remove tags from mandatory event arguments
diff --git a/src/Symfony/Component/Mailer/Bridge/Brevo/Webhook/BrevoRequestParser.php b/src/Symfony/Component/Mailer/Bridge/Brevo/Webhook/BrevoRequestParser.php
@@ -49,7 +49,6 @@ protected function doParse(Request $request, #[\SensitiveParameter] string $secr
             || !isset($content['email'])
             || !isset($content['message-id'])
             || !isset($content['ts_event'])
-            || !isset($content['tags'])
         ) {
             throw new RejectWebhookException(406, 'Payload is malformed.');
         }

Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,6 @@ protected function doParse(Request $request, #[\SensitiveParameter] string $secr`
`49`	`49`	`\|\| !isset($content['email'])`
`50`	`50`	`\|\| !isset($content['message-id'])`
`51`	`51`	`\|\| !isset($content['ts_event'])`
`52`		`- \|\| !isset($content['tags'])`
`53`	`52`	`) {`
`54`	`53`	`throw new RejectWebhookException(406, 'Payload is malformed.');`
`55`	`54`	`}`