feat: adapt webhook payload format

Signed-off-by: Jana Peper <[email protected]>

Author: janepie

apps/webhook_listeners/lib/BackgroundJobs/WebhookCall.php

@@ -46,7 +46,7 @@ protected function run($argument): void {
 		$client = $this->clientService->newClient();
 
 		// adding Ephemeral auth tokens to the call
-		$data['tokens'] = $this->tokenService->getTokens($webhookListener, $data['user']['uid'] ?? null);
+		$data['authentication'] = $this->tokenService->getTokens($webhookListener, $data['user']['uid'] ?? null);
 		$options = [
 			'verify' => $this->certificateManager->getAbsoluteBundlePath(),
 			'headers' => $webhookListener->getHeaders() ?? [],

apps/webhook_listeners/lib/Service/TokenService.php

@@ -14,6 +14,7 @@
 use OCA\WebhookListeners\Db\WebhookListener;
 use OCP\AppFramework\Utility\ITimeFactory;
 use OCP\Authentication\Token\IToken;
+use OCP\IURLGenerator;
 use OCP\IUserManager;
 use OCP\L10N\IFactory;
 use OCP\Security\ISecureRandom;
@@ -22,6 +23,7 @@
 class TokenService {
 	public function __construct(
 		private IProvider $tokenProvider,
+		private IURLGenerator $urlGenerator,
 		private ISecureRandom $random,
 		private EphemeralTokenMapper $tokenMapper,
 		private LoggerInterface $logger,
@@ -35,20 +37,21 @@ public function __construct(
 	 * creates an array which includes two arrays of tokens: 'user_ids' and 'user_roles'
 	 * The array ['user_ids' => ['jane', 'bob'], 'user_roles' => ['owner', 'trigger']]
 	 * as requested tokens in the registered webhook produces a result like
-	 * ['user_ids' => [['jane' => 'abcdtokenabcd1'], ['bob','=> 'abcdtokenabcd2']], 'user_roles' => ['owner' => ['admin' => 'abcdtokenabcd3'], 'trigger' => ['user1' => 'abcdtokenabcd4']]]
+	 * ['user_ids' => [['jane' => 'abcdtokenabcd1'], ['bob','=> 'abcdtokenabcd2']], 'owner' => ['admin' => 'abcdtokenabcd3'], 'trigger' => ['user1' => 'abcdtokenabcd4']]
 	 * Created auth tokens are valid for 1 hour.
 	 *
 	 * @param WebhookListener $webhookListener
 	 * @param ?string $triggerUserId the user that triggered the webhook call
 	 * @return array{user_ids?:array<string,string>,user_roles?:array{owner?:array<string,string>,trigger?:array<string,string>}}
 	 */
 	public function getTokens(WebhookListener $webhookListener, ?string $triggerUserId): array {
-		$tokens = [
-			'user_ids' => [],
-			'user_roles' => [],
-		];
+		$tokens = [];
+
 		$tokenNeeded = $webhookListener->getTokenNeeded();
 		if (isset($tokenNeeded['user_ids'])) {
+			$tokens = [
+				'user_ids' => [],
+			];
 			foreach ($tokenNeeded['user_ids'] as $userId) {
 				try {
 					$tokens['user_ids'][$userId] = $this->createEphemeralToken($userId);
@@ -67,17 +70,19 @@ public function getTokens(WebhookListener $webhookListener, ?string $triggerUser
 						if (is_null($ownerId)) { // no owner uid available
 							break;
 						}
-						$tokens['user_roles']['owner'] = [
-							$ownerId => $this->createEphemeralToken($ownerId)
+						$tokens['owner'] = [
+							'token' => $this->createEphemeralToken($ownerId),
+							'baseUrl' => $this->urlGenerator->getBaseUrl()
 						];
 						break;
 					case 'trigger':
 						// token for the person who triggered the webhook
 						if (is_null($triggerUserId)) { // no trigger uid available
 							break;
 						}
-						$tokens['user_roles']['trigger'] = [
-							$triggerUserId => $this->createEphemeralToken($triggerUserId)
+						$tokens['trigger'] = [
+							'token' => $this->createEphemeralToken($triggerUserId),
+							'baseUrl' => $this->urlGenerator->getBaseUrl()
 						];
 						break;
 					default: