feat(preset): add lexicon entry for custom share tokens

Signed-off-by: Maxence Lange <[email protected]>

Author: ArtificialOwl

core/AppInfo/ConfigLexicon.php

@@ -20,6 +20,7 @@
  */
 class ConfigLexicon implements ILexicon {
 	public const SHAREAPI_ALLOW_FEDERATION_ON_PUBLIC_SHARES = 'shareapi_allow_federation_on_public_shares';
+	public const SHARE_CUSTOM_TOKEN = 'shareapi_allow_custom_tokens';
 
 	public function getStrictness(): Strictness {
 		return Strictness::IGNORE;
@@ -34,6 +35,20 @@ public function getAppConfigs(): array {
 				definition: 'adds share permission to public shares to allow adding them to your Nextcloud (federation)',
 				lazy: true,
 			),
+			new Entry(
+				key: self::SHARE_CUSTOM_TOKEN,
+				type: ValueType::BOOL,
+				defaultRaw: fn (Preset $p): bool => match ($p) {
+					Preset::FAMILY, Preset::PRIVATE => true,
+					default => false,
+				},
+				definition: [
+					'definition' => 'Allow users to set custom share link tokens',
+					'note' => 'Shares with custom tokens will continue to be accessible after this setting has been disabled',
+					'warning' => 'Shares with guessable tokens may be accessed easily'
+				],
+				lazy: true,
+			),
 		];
 	}
 

lib/private/Share20/Manager.php

@@ -7,6 +7,7 @@
  */
 namespace OC\Share20;
 
+use OC\Core\AppInfo\ConfigLexicon;
 use OC\Files\Mount\MoveableMount;
 use OC\KnownUser\KnownUserService;
 use OC\Share20\Exception\ProviderException;
@@ -1937,7 +1938,7 @@ public function ignoreSecondDisplayName(): bool {
 	}
 
 	public function allowCustomTokens(): bool {
-		return $this->appConfig->getValueBool('core', 'shareapi_allow_custom_tokens', false);
+		return $this->appConfig->getValueBool('core', ConfigLexicon::SHARE_CUSTOM_TOKEN);
 	}
 
 	public function allowViewWithoutDownload(): bool {

lib/public/Config/Lexicon/Entry.php

@@ -21,7 +21,7 @@ class Entry {
 	/** @since 32.0.0 */
 	public const RENAME_INVERT_BOOLEAN = 1;
 
-	private string $definition = '';
+	private array|string $definition = '';
 	private ?string $default = null;
 
 	/**
@@ -41,7 +41,7 @@ public function __construct(
 		private readonly string $key,
 		private readonly ValueType $type,
 		private null|string|int|float|bool|array|Closure $defaultRaw = null,
-		string $definition = '',
+		string|array $definition = '',
 		private readonly bool $lazy = false,
 		private readonly int $flags = 0,
 		private readonly bool $deprecated = false,
@@ -183,7 +183,31 @@ public function convertToString(string|int|float|bool|array $entry): string {
 	 * @since 32.0.0
 	 */
 	public function getDefinition(): string {
-		return $this->definition;
+		if (is_string($this->definition)) {
+			return $this->definition;
+		}
+
+		return (string)($this->definition['definition'] ?? '');
+	}
+
+	/**
+	 * returns eventual note
+	 *
+	 * @return string
+	 * @since 32.0.0
+	 */
+	public function getNote(): string {
+		return (string)($this->definition['note'] ?? '');
+	}
+
+	/**
+	 * returns eventual warning
+	 *
+	 * @return string
+	 * @since 32.0.0
+	 */
+	public function getWarning(): string {
+		return (string)($this->definition['warning'] ?? '');
 	}
 
 	/**

lib/unstable/Config/Lexicon/ConfigLexiconEntry.php

@@ -28,13 +28,13 @@ class ConfigLexiconEntry {
 	 */
 	public const RENAME_INVERT_BOOLEAN = 1;
 
-	private string $definition = '';
+	private array|string $definition = '';
 	private ?string $default = null;
 
 	/**
 	 * @param string $key config key, can only contain alphanumerical chars and -._
 	 * @param ValueType $type type of config value
-	 * @param string $definition optional description of config key available when using occ command
+	 * @param array|string $definition optional description of config key available when using occ command
 	 * @param bool $lazy set config value as lazy
 	 * @param int $flags set flags
 	 * @param string|null $rename previous config key to migrate config value from
@@ -50,7 +50,7 @@ public function __construct(
 		private readonly string $key,
 		private readonly ValueType $type,
 		private null|string|int|float|bool|array|Closure $defaultRaw = null,
-		string $definition = '',
+		string|array $definition = '',
 		private readonly bool $lazy = false,
 		private readonly int $flags = 0,
 		private readonly bool $deprecated = false,
@@ -218,9 +218,34 @@ public function convertToString(string|int|float|bool|array $entry): string {
 	 * @see \OCP\Config\Lexicon\Entry
 	 */
 	public function getDefinition(): string {
-		return $this->definition;
+		if (is_string($this->definition)) {
+			return $this->definition;
+		}
+
+		return (string)($this->definition['definition'] ?? '');
+	}
+
+	/**
+	 * returns eventual note
+	 *
+	 * @return string
+	 * @experimental 32.0.0
+	 */
+	public function getNote(): string {
+		return (string)($this->definition['note'] ?? '');
 	}
 
+	/**
+	 * returns eventual warning
+	 *
+	 * @return string
+	 * @experimental 32.0.0
+	 */
+	public function getWarning(): string {
+		return (string)($this->definition['warning'] ?? '');
+	}
+
+
 	/**
 	 * returns if config key is set as lazy
 	 *