Merge pull request #21 from emchooo/fix-event-type

rennokki · web-flow · commit 2c3979ee317a · 2021-10-13T20:35:06.000+03:00
[fix] Add check for eventType or notificationType in AWS SES handler
diff --git a/src/Http/Controllers/SesWebhook.php b/src/Http/Controllers/SesWebhook.php
@@ -19,7 +19,8 @@ protected function onNotification(array $snsMessage, Request $request): void
     {
         $decodedMessage = json_decode($snsMessage['Message'], true);
 
-        $eventType = $decodedMessage['eventType'] ?? null;
+        $typeKey = array_key_exists('eventType', $decodedMessage) ? 'eventType' : 'notificationType';
+        $eventType = $decodedMessage[$typeKey] ?? null;
 
         $methodToCall = 'on'.Str::studly($eventType);
 

Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,8 @@ protected function onNotification(array $snsMessage, Request $request): void`
`19`	`19`	`{`
`20`	`20`	`$decodedMessage = json_decode($snsMessage['Message'], true);`
`21`	`21`
`22`		`- $eventType = $decodedMessage['eventType'] ?? null;`
	`22`	`+ $typeKey = array_key_exists('eventType', $decodedMessage) ? 'eventType' : 'notificationType';`
	`23`	`+ $eventType = $decodedMessage[$typeKey] ?? null;`
`23`	`24`
`24`	`25`	`$methodToCall = 'on'.Str::studly($eventType);`
`25`	`26`