Merge pull request #678 from nextcloud/bugfix/noid/allow-to-grant-log-access

Allow to grant log access

Author: nickvergessen

lib/Controller/LogController.php

@@ -49,6 +49,7 @@ public function __construct($appName,
 	}
 
 	/**
+	 * @AuthorizedAdminSetting(settings=OCA\LogReader\Settings\Admin)
 	 * @param int $count
 	 * @param int $offset
 	 * @param string $levels
@@ -71,6 +72,7 @@ private function getLastItem($levels) {
 	}
 
 	/**
+	 * @AuthorizedAdminSetting(settings=OCA\LogReader\Settings\Admin)
 	 * @brief polls for a new log message since $lastReqId.
 	 * This method will sleep for maximum 20 seconds before returning an empty
 	 * result.
@@ -121,6 +123,7 @@ public function poll(string $lastReqId, string $levels = '11111'): JSONResponse
 	}
 
 	/**
+	 * @AuthorizedAdminSetting(settings=OCA\LogReader\Settings\Admin)
 	 * @param string $query
 	 * @param int $count
 	 * @param int $offset
@@ -138,11 +141,17 @@ public function search($query = '', $count = 50, $offset = 0, $levels = '11111')
 		return $this->responseFromIterator($iterator, $count, $offset);
 	}
 
-	public function getLevels() {
+	/**
+	 * @AuthorizedAdminSetting(settings=OCA\LogReader\Settings\Admin)
+	 */
+	public function getLevels(): JSONResponse {
 		return new JSONResponse($this->config->getAppValue('logreader', 'levels', '11111'));
 	}
 
-	public function getSettings() {
+	/**
+	 * @AuthorizedAdminSetting(settings=OCA\LogReader\Settings\Admin)
+	 */
+	public function getSettings(): JSONResponse {
 		return new JSONResponse([
 			'levels' => $this->config->getAppValue('logreader', 'levels', '11111'),
 			'dateformat' => $this->config->getSystemValue('logdateformat', \DateTime::ISO8601),
@@ -153,20 +162,25 @@ public function getSettings() {
 	}
 
 	/**
+	 * @AuthorizedAdminSetting(settings=OCA\LogReader\Settings\Admin)
 	 * @param bool $relative
 	 */
 	public function setRelative($relative) {
 		$this->config->setAppValue('logreader', 'relativedates', $relative);
 	}
 
 	/**
+	 * @AuthorizedAdminSetting(settings=OCA\LogReader\Settings\Admin)
 	 * @param bool $live
 	 */
 	public function setLive($live) {
 		$this->config->setAppValue('logreader', 'live', $live);
 	}
 
-	public function setLevels($levels) {
+	/**
+	 * @AuthorizedAdminSetting(settings=OCA\LogReader\Settings\Admin)
+	 */
+	public function setLevels(string $levels): int {
 		$intLevels = array_map('intval', str_split($levels));
 		$minLevel = 4;
 		foreach ($intLevels as $level => $log) {

lib/Settings/Admin.php

@@ -22,9 +22,16 @@
 namespace OCA\LogReader\Settings;
 
 use OCP\AppFramework\Http\TemplateResponse;
-use OCP\Settings\ISettings;
+use OCP\IL10N;
+use OCP\Settings\IDelegatedSettings;
 
-class Admin implements ISettings {
+class Admin implements IDelegatedSettings {
+	/** @var IL10N */
+	private $l;
+
+	public function __construct(IL10N $l) {
+		$this->l = $l;
+	}
 	/**
 	 * @return TemplateResponse
 	 */
@@ -52,4 +59,12 @@ public function getSection() {
 	public function getPriority() {
 		return 90;
 	}
+
+	public function getName(): ?string {
+		return null;
+	}
+
+	public function getAuthorizedAppConfig(): array {
+		return [];
+	}
 }