fix(push): Use fallback keys that look like a real one

Signed-off-by: Joas Schilling <coding@schilljs.com> [skip ci]
nextcloud · Oct 21, 2024 · 43b51e9 · 43b51e9
1 parent aa17c3e
commit 43b51e9
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/lib/Push.php b/lib/Push.php
@@ -30,6 +30,7 @@
 use OCP\Notification\IManager as INotificationManager;
 use OCP\Notification\IncompleteParsedNotificationException;
 use OCP\Notification\INotification;
+use OCP\Security\ISecureRandom;
 use OCP\UserStatus\IManager as IUserStatusManager;
 use OCP\UserStatus\IUserStatus;
 use OCP\Util;
@@ -474,7 +475,11 @@ protected function sendNotificationsToProxies(): void {
 		if ($subscriptionAwareServer === 'https://push-notifications.nextcloud.com') {
 			$subscriptionKey = $this->config->getAppValue('support', 'subscription_key');
 		} else {
-			$subscriptionKey = $this->config->getSystemValueString('instanceid');
+			$subscriptionKey = $this->config->getAppValue(Application::APP_ID, 'push_subscription_key');
+			if ($subscriptionKey === '') {
+				$subscriptionKey = $this->createPushSubscriptionKey();
+				$this->config->setAppValue(Application::APP_ID, 'push_subscription_key', $subscriptionKey);
+			}
 		}
 
 		$client = $this->clientService->newClient();
@@ -781,4 +786,9 @@ protected function deletePushTokenByDeviceIdentifier(string $deviceIdentifier):
 	protected function createFakeUserObject(string $userId): IUser {
 		return new FakeUser($userId);
 	}
+
+	protected function createPushSubscriptionKey(): string {
+		$key = $this->random->generate(25, ISecureRandom::CHAR_ALPHANUMERIC);
+		return implode('-', str_split($key, 5));
+	}
 }
diff --git a/tests/Unit/PushTest.php b/tests/Unit/PushTest.php
@@ -26,6 +26,7 @@
 use OCP\L10N\IFactory;
 use OCP\Notification\IManager as INotificationManager;
 use OCP\Notification\INotification;
+use OCP\Security\ISecureRandom;
 use OCP\UserStatus\IManager as IUserStatusManager;
 use PHPUnit\Framework\MockObject\MockObject;
 use Psr\Http\Message\ResponseInterface;
@@ -78,6 +79,7 @@ protected function setUp(): void {
 		$this->userStatusManager = $this->createMock(IUserStatusManager::class);
 		$this->l10nFactory = $this->createMock(IFactory::class);
 		$this->timeFactory = $this->createMock(ITimeFactory::class);
+		$this->random = $this->createMock(ISecureRandom::class);
 		$this->logger = $this->createMock(LoggerInterface::class);
 
 		$this->cacheFactory->method('createDistributed')
@@ -103,6 +105,7 @@ protected function getPush(array $methods = []) {
 					$this->userStatusManager,
 					$this->l10nFactory,
 					$this->timeFactory,
+					$this->random,
 					$this->logger,
 				])
 				->setMethods($methods)