Add option to skip diagnostic checks (#1558)

.env.example

@@ -89,3 +89,6 @@ MAIL_FROM_ADDRESS=
 #  - `*`: any proxy
 #  - <ip address>[,<ip address>]: a comma-seperated list of IP addresses
 TRUSTED_PROXIES=null
+
+# Comma-separated list of class names of diagnostics checks that should be skipped.
+#SKIP_DIAGNOSTICS_CHECKS= 

app/Actions/Diagnostics/Errors.php

@@ -16,9 +16,11 @@ public function __construct(DiagnosticsChecksFactory $diagnosticsChecksFactory)
 	/**
 	 * Return the list of error which are currently breaking Lychee.
 	 *
+	 * @param string[] $skip class names of checks that will be skipped
+	 *
 	 * @return string[] array of messages
 	 */
-	public function get(): array
+	public function get(array $skip = []): array
 	{
 		/** @var string[] $errors */
 		$errors = [];
@@ -28,7 +30,10 @@ public function get(): array
 		$checks = $this->diagnosticsChecksFactory->makeAll();
 
 		foreach ($checks as $check) {
-			$check->check($errors);
+			$check_name = (new \ReflectionClass($check))->getShortName();
+			if (!in_array($check_name, $skip, true)) {
+				$check->check($errors);
+			}
 		}
 		// @codeCoverageIgnoreEnd
 

app/Console/Commands/Diagnostics.php

@@ -24,7 +24,8 @@ class Diagnostics extends Command
 	 *
 	 * @var string
 	 */
-	protected $signature = 'lychee:diagnostics';
+	protected $signature = 'lychee:diagnostics
+	                {--skip=* : Skip certain diagnostics check, overrides SKIP_DIAGNOSTICS_CHECKS config}';
 
 	/**
 	 * The console command description.
@@ -73,10 +74,17 @@ private function block(string $str, array $array): void
 	 */
 	public function handle(): int
 	{
+		/** @var string[] $skip_diagnostics */
+		$skip_diagnostics = config('app.skip_diagnostics_checks');
+		/** @var string[] $options */
+		$options = $this->option('skip');
+		if (sizeof($options) > 0) {
+			$skip_diagnostics = $options;
+		}
 		try {
 			$this->line('');
 			$this->line('');
-			$this->block('Diagnostics', resolve(Errors::class)->get());
+			$this->block('Diagnostics', resolve(Errors::class)->get($skip_diagnostics));
 			$this->line('');
 			$this->block('System Information', resolve(Info::class)->get());
 			$this->line('');

app/Http/Controllers/Administration/DiagnosticsController.php

@@ -49,7 +49,7 @@ public function get(Errors $checkErrors, Info $collectInfo, Configuration $confi
 	{
 		$authorized = $this->isAuthorized();
 
-		return new DiagnosticInfo($checkErrors->get(), $authorized ? $collectInfo->get() : [self::ERROR_MSG], $authorized ? $config->get() : [self::ERROR_MSG], $checkUpdate->getCode());
+		return new DiagnosticInfo($checkErrors->get(config('app.skip_diagnostics_checks')), $authorized ? $collectInfo->get() : [self::ERROR_MSG], $authorized ? $config->get() : [self::ERROR_MSG], $checkUpdate->getCode());
 	}
 
 	/**

config/app.php

@@ -120,6 +120,16 @@
 
 	'db_log_sql' => (bool) env('DB_LOG_SQL', false),
 
+	/*
+	|--------------------------------------------------------------------------
+	| Skip diagnostics checks
+	|--------------------------------------------------------------------------
+	|
+	| Allows to define class names of diagnostics checks that will be skipped.
+	|
+	*/
+	'skip_diagnostics_checks' => explode(',', env('SKIP_DIAGNOSTICS_CHECKS', '')),
+
 	/*
 	|--------------------------------------------------------------------------
 	| Encryption Key