fix: move endpoint back to its own controller

Author: davidmrc6

appinfo/routes.php

@@ -113,7 +113,7 @@
 		['name' => 'termsagreement_api#agree', 'url' => '/api/1.0/termsagreement/agree/{version}', 'verb' => 'POST'],
 		['name' => 'termsagreement_api#isAgreed', 'url' => '/api/1.0/termsagreement/isagreed/{version}', 'verb' => 'GET'],
 
-		['name' => 'status_api#releases', 'url' => '/api/1.0/releases', 'verb' => 'GET'],
+		['name' => 'releases_api#index', 'url' => '/api/1.0/releases', 'verb' => 'GET'],
 
 		[
 			'name' => 'setting_groups_management#preflighted_cors',

lib/Controller/ReleasesApiController.php

@@ -0,0 +1,79 @@
+<?php
+
+/**
+ * @copyright Copyright (c) 2026 Sendent B.V.
+ *
+ * @author Sendent B.V. <info@sendent.com>
+ *
+ * @license GNU AGPL version 3 or any later version
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ */
+
+namespace OCA\Sendent\Controller;
+
+use OCP\AppFramework\ApiController;
+use OCP\AppFramework\Http\Attribute\NoAdminRequired;
+use OCP\AppFramework\Http\Attribute\NoCSRFRequired;
+use OCP\AppFramework\Http\DataResponse;
+use OCP\Http\Client\IClientService;
+use OCP\IRequest;
+use Psr\Log\LoggerInterface;
+
+class ReleasesApiController extends ApiController {
+	private const BASE_URL = 'https://releasesapp.com/api/entries/latest/';
+
+	private const WORKSPACES = [
+		'outlook-cross-platform' => 'dc9b6f11-139c-46ad-83a7-efca461ef0d0',
+		'ms-teams' => '6880e895-ba93-4433-9729-371ea9dc0ac1',
+		'outlook-windows' => '13f7862f-8df4-4106-a818-4d5a82f6fe50',
+	];
+
+	private $httpClient;
+	private $logger;
+
+	public function __construct(
+		$appName,
+		IRequest $request,
+		IClientService $clientService,
+		LoggerInterface $logger,
+	) {
+		parent::__construct($appName, $request);
+		$this->httpClient = $clientService->newClient();
+		$this->logger = $logger;
+	}
+
+	/**
+	 * Fetches the latest release entry for all products.
+	 *
+	 * @return DataResponse
+	 */
+	#[NoAdminRequired]
+	#[NoCSRFRequired]
+	public function index(): DataResponse {
+		$results = [];
+		foreach (self::WORKSPACES as $slug => $uuid) {
+			try {
+				$response = $this->httpClient->get(self::BASE_URL . $uuid);
+				$data = json_decode($response->getBody(), true);
+				if ($data) {
+					$results[$slug] = $data;
+				}
+			} catch (\Exception $e) {
+				$this->logger->warning('Failed to fetch release for ' . $slug . ': ' . $e->getMessage());
+			}
+		}
+		return new DataResponse($results);
+	}
+}

lib/Controller/StatusApiController.php

@@ -32,9 +32,7 @@
 use OCP\AppFramework\Http\Attribute\NoAdminRequired;
 use OCP\AppFramework\Http\Attribute\NoCSRFRequired;
 use OCP\AppFramework\Http\DataResponse;
-use OCP\Http\Client\IClientService;
 use OCP\IRequest;
-use Psr\Log\LoggerInterface;
 
 class StatusApiController extends ApiController {
 	/** @var IAppManager */
@@ -44,15 +42,6 @@ class StatusApiController extends ApiController {
 	private $licensemanager;
 	private $appVersionClient;
 	private $licenseservice;
-	private $httpClient;
-	private $logger;
-
-	private const RELEASES_BASE_URL = 'https://releasesapp.com/api/entries/latest/';
-	private const RELEASE_WORKSPACES = [
-		'outlook-cross-platform' => 'dc9b6f11-139c-46ad-83a7-efca461ef0d0',
-		'ms-teams' => '6880e895-ba93-4433-9729-371ea9dc0ac1',
-		'outlook-windows' => '13f7862f-8df4-4106-a818-4d5a82f6fe50',
-	];
 
 	public function __construct(
 		$appName,
@@ -62,17 +51,13 @@ public function __construct(
 		LicenseManager $licensemanager,
 		AppVersionHttpClient $appVersionClient,
 		LicenseService $licenseservice,
-		IClientService $clientService,
-		LoggerInterface $logger,
 	) {
 		parent::__construct($appName, $request);
 		$this->appManager = $appManager;
 		$this->userId = $userId;
 		$this->appVersionClient = $appVersionClient;
 		$this->licensemanager = $licensemanager;
 		$this->licenseservice = $licenseservice;
-		$this->httpClient = $clientService->newClient();
-		$this->logger = $logger;
 	}
 	/**
 	 * Get the status of the user's license
@@ -147,27 +132,4 @@ public function index(): DataResponse {
 		// Returns license status
 		return new DataResponse($statusobj);
 	}
-
-	/**
-	 * Fetches the latest release entry for all products from releasesapp.com
-	 *
-	 * @return DataResponse
-	 */
-	#[NoAdminRequired]
-	#[NoCSRFRequired]
-	public function releases(): DataResponse {
-		$results = [];
-		foreach (self::RELEASE_WORKSPACES as $slug => $uuid) {
-			try {
-				$response = $this->httpClient->get(self::RELEASES_BASE_URL . $uuid);
-				$data = json_decode($response->getBody(), true);
-				if ($data) {
-					$results[$slug] = $data;
-				}
-			} catch (\Exception $e) {
-				$this->logger->warning('Failed to fetch release for ' . $slug . ': ' . $e->getMessage());
-			}
-		}
-		return new DataResponse($results);
-	}
 }