Merge pull request #53 from iMattPro/fixes

Trivial updates

Author: iMattPro

README.md

@@ -37,16 +37,16 @@ Testing push notifications necessitates user-to-user interactions to observe the
 
 1. **User Account Setup:**
 	- Create at least two distinct user accounts for testing purposes.
-    - In each user account's notifications preferences, assign web push notifications and subscribe to them to receive Push Notifications.
+    - In user1's account notifications preferences, assign web push notifications and subscribe to receive Push Notifications.
 
 2. **Message, Quote, or Reply Interaction:**
-	- Initiate a user-to-user interaction by performing one of the following actions using "User Account 1":
-		- **Private Message:** Send a direct message from "User Account 1" to "User Account 2".
-		- **Quote:** Quote a post or message authored by "User Account 2" using "User Account 1".
-		- **Reply:** Respond to a post or message authored by "User Account 2" using "User Account 1".
+	- Initiate a user-to-user interaction by performing one of the following actions using "User Account 2":
+		- **Private Message:** Send a direct message from "User Account 2" to "User Account 1".
+		- **Quote:** Quote a post or message authored by "User Account 1" using "User Account 2".
+		- **Reply:** Respond to a post or message authored by "User Account 1" using "User Account 2".
 
 3. **Observing Push Notifications:**
-	- Once the interaction is performed from "User Account 1" to engage with "User Account 2," you promptly should see a notification for "User Account 2."
+	- Once the interaction is performed from "User Account 2" to engage with "User Account 1," you promptly should see a notification for "User Account 1."
 
 4. **Caveats for Local Testing**
     - Local testing of Push Notifications only works from a `localhost` address or if your local server has a secure SSL certificate.

notification/method/webpush.php

@@ -495,13 +495,19 @@ protected function get_board_url()
 	 * @param string $endpoint
 	 *
 	 * @return void
-	 * @throws \Exception
 	 */
 	protected function set_endpoint_padding(\Minishlink\WebPush\WebPush $web_push, string $endpoint): void
 	{
 		if (strpos($endpoint, 'mozilla.com') || strpos($endpoint, 'mozaws.net'))
 		{
-			$web_push->setAutomaticPadding(self::MOZILLA_FALLBACK_PADDING);
+			try
+			{
+				$web_push->setAutomaticPadding(self::MOZILLA_FALLBACK_PADDING);
+			}
+			catch (\Exception $e)
+			{
+				// This shouldn't happen since we won't pass padding length outside limits
+			}
 		}
 	}
 }

tests/notification/notification_method_webpush_test.php

@@ -651,6 +651,47 @@ public function test_prune_notifications($notification_type, $post_data, $expect
 		$this->assertCount(0, $cur_notifications, 'Assert that no notifications have been pruned');
 	}
 
+	public function data_set_endpoint_padding(): array
+	{
+		return [
+			[
+				'foo.mozilla.com',
+				webpush::MOZILLA_FALLBACK_PADDING
+			],
+			[
+				'foo.mozaws.net',
+				webpush::MOZILLA_FALLBACK_PADDING
+			],
+			[
+				'foo.android.googleapis.com',
+				\Minishlink\WebPush\Encryption::MAX_COMPATIBILITY_PAYLOAD_LENGTH,
+			],
+		];
+	}
+
+	/**
+	 * @dataProvider data_set_endpoint_padding
+	 */
+	public function test_set_endpoint_padding($endpoint, $expected_padding): void
+	{
+		$web_push_reflection = new \ReflectionMethod($this->notification_method_webpush, 'set_endpoint_padding');
+		$web_push_reflection->setAccessible(true);
+
+		$auth = [
+			'VAPID' => [
+				'subject' => generate_board_url(),
+				'publicKey' => $this->config['wpn_webpush_vapid_public'],
+				'privateKey' => $this->config['wpn_webpush_vapid_private'],
+			],
+		];
+
+		$web_push = new \Minishlink\WebPush\WebPush($auth);
+
+		$this->assertEquals(\Minishlink\WebPush\Encryption::MAX_COMPATIBILITY_PAYLOAD_LENGTH, $web_push->getAutomaticPadding());
+		$web_push_reflection->invoke($this->notification_method_webpush, $web_push, $endpoint);
+		$this->assertEquals($expected_padding, $web_push->getAutomaticPadding());
+	}
+
 	protected function create_subscription_for_user($user_id, bool $invalidate_endpoint = false): array
 	{
 		$client = new \GuzzleHttp\Client();