make sure the record size is <= 4078 bytes

Author: Minishlink

src/WebPush.php

@@ -82,6 +82,10 @@ public function sendNotification($endpoint, $payload = null, $userPublicKey = nu
             throw new \ErrorException('The API has changed: sendNotification now takes the optional user auth token as parameter.');
         }
 
+        if(isset($payload) && strlen($payload) > 4078) {
+            throw new \ErrorException('Size of payload must not be greater than 4078 octets.');
+        }
+
         // sort notification by server type
         $type = $this->sortEndpoint($endpoint);
         $this->notificationsByServerType[$type][] = new Notification($endpoint, $payload, $userPublicKey, $userAuthToken);

tests/WebPushTest.php

@@ -83,6 +83,18 @@ public function testSendNotificationWithOldAPI()
         );
     }
 
+    public function testSendNotificationWithTooBigPayload()
+    {
+        $this->setExpectedException('ErrorException', 'Size of payload must not be greater than 4078 octets.');
+        $this->webPush->sendNotification(
+            $this->endpoints['standard'],
+            str_repeat('test', 1020),
+            $this->keys['standard'],
+            null,
+            true
+        );
+    }
+
     public function testSendNotifications()
     {
         foreach($this->endpoints as $endpoint) {