fix: remove redundant db column

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

Author: janepie

apps/webhook_listeners/lib/Db/EphemeralToken.php

@@ -13,37 +13,30 @@
 
 /**
  * @method int getTokenId()
- * @method string getToken()
  * @method ?string getUserId()
  * @method int getCreatedAt()
  * @psalm-suppress PropertyNotSetInConstructor
  */
 class EphemeralToken extends Entity implements \JsonSerializable {
 	/**
-	 * @var int id of the token that was created for a webhook
+	 * @var int id of the token in the oc_authtoken db table
 	 */
 	protected $tokenId;
 
-	/**
-	 * @var string the token
-	 */
-	protected $token;
-
 	/**
 	 * @var ?string id of the user wich the token belongs to
 	 * @psalm-suppress PropertyNotSetInConstructor
 	 */
 	protected $userId = null;
 
 	/**
-	 * @var int
+	 * @var int token creation timestamp
 	 * @psalm-suppress PropertyNotSetInConstructor
 	 */
 	protected $createdAt;
 
 	public function __construct() {
 		$this->addType('tokenId', 'integer');
-		$this->addType('token', 'string');
 		$this->addType('userId', 'string');
 		$this->addType('createdAt', 'integer');
 	}

apps/webhook_listeners/lib/Db/EphemeralTokenMapper.php

@@ -39,7 +39,7 @@ public function __construct(
 	/**
 	 * @throws DoesNotExistException
 	 * @throws MultipleObjectsReturnedException
-	 * @throws Exception
+	 * @throws \Exception
 	 */
 	public function getById(int $id): EphemeralToken {
 		$qb = $this->db->getQueryBuilder();
@@ -52,7 +52,7 @@ public function getById(int $id): EphemeralToken {
 	}
 
 	/**
-	 * @throws Exception
+	 * @throws \Exception
 	 * @return EphemeralToken[]
 	 */
 	public function getAll(): array {
@@ -75,18 +75,16 @@ public function getOlderThan($olderThan): array {
 	}
 
 	/**
-	 * @throws Exception
+	 * @throws \Exception
 	 */
 	public function addEphemeralToken(
 		int $tokenId,
-		string $token,
 		?string $userId,
 		int $createdAt,
 	): EphemeralToken {
 		$tempToken = EphemeralToken::fromParams(
 			[
 				'tokenId' => $tokenId,
-				'token' => $token,
 				'userId' => $userId,
 				'createdAt' => $createdAt,
 			]
@@ -95,7 +93,7 @@ public function addEphemeralToken(
 	}
 
 	/**
-	 * @throws Exception
+	 * @throws \Exception
 	 */
 	public function deleteByTokenId(int $tokenId): bool {
 		$qb = $this->db->getQueryBuilder();
@@ -112,7 +110,7 @@ public function invalidateOldTokens(int $token_lifetime = self::TOKEN_LIFETIME)
 
 		$this->logger->debug('Invalidating ephemeral webhook tokens older than ' . date('c', $olderThan), ['app' => 'webhook_listeners']);
 		foreach ($tokensToDelete as $token) {
-			$this->tokenMapper->invalidate($token->getToken()); // delete token itself
+			$this->tokenMapper->invalidate($this->tokenMapper->getTokenById($token->getTokenId())->getToken()); // delete token itself
 			$this->deleteByTokenId($token->getTokenId()); // delete db row in webhook_tokens
 		}
 	}

apps/webhook_listeners/lib/Migration/Version1500Date20251007130000.php

@@ -51,10 +51,6 @@ public function changeSchema(IOutput $output, Closure $schemaClosure, array $opt
 				'length' => 4,
 				'unsigned' => true,
 			]);
-			$table->addColumn('token', Types::STRING, [
-				'notnull' => false,
-				'length' => 200,
-			]);
 			$table->addColumn('user_id', Types::STRING, [
 				'notnull' => false,
 				'length' => 64,

apps/webhook_listeners/lib/Service/TokenService.php

@@ -104,7 +104,6 @@ private function createEphemeralToken(string $userId): string {
 		}
 		$this->tokenMapper->addEphemeralToken(
 			$deviceToken->getId(),
-			$deviceToken->getToken(),
 			$userId,
 			$this->time->getTime());
 		return $token;