refactor: Replace security annotations with attributes in LogController

Signed-off-by: Josh <[email protected]>

Author: joshtrichards

lib/Controller/LogController.php

@@ -25,8 +25,10 @@
 use OCA\LogReader\Log\LogIteratorFactory;
 use OCA\LogReader\Log\SearchFilter;
 use OCA\LogReader\Service\SettingsService;
+use OCA\LogReader\Settings\Admin;
 use OCP\AppFramework\Controller;
 use OCP\AppFramework\Http;
+use OCP\AppFramework\Http\Attribute\AuthorizedAdminSetting;
 use OCP\AppFramework\Http\JSONResponse;
 use OCP\IRequest;
 use Psr\Log\LoggerInterface;
@@ -48,12 +50,12 @@ public function __construct($appName,
 	}
 
 	/**
-	 * @AuthorizedAdminSetting(settings=OCA\LogReader\Settings\Admin)
 	 * @param string $query
 	 * @param int $count
 	 * @param int $offset
 	 * @return JSONResponse
 	 */
+	#[AuthorizedAdminSetting(settings: Admin::class)]
 	public function get($query = '', $count = 50, $offset = 0): JSONResponse {
 		$logType = $this->settingsService->getLoggingType();
 		// we only support web access when `log_type` is set to `file` (the default)
@@ -86,7 +88,6 @@ private function getLastItem() {
 	}
 
 	/**
-	 * @AuthorizedAdminSetting(settings=OCA\LogReader\Settings\Admin)
 	 * @brief Use to poll for new log messages since $lastReqId.
 	 *
 	 * @note There is a possible race condition, when the user loads the
@@ -99,6 +100,7 @@ private function getLastItem() {
 	 *  will work in some cases but not when there are more than 50 messages of that
 	 *  request.
 	 */
+	#[AuthorizedAdminSetting(settings: Admin::class)]
 	public function poll(string $lastReqId): JSONResponse {
 		$logType = $this->settingsService->getLoggingType();
 		// we only support web access when `log_type` is set to `file` (the default)