Merge pull request #526 from nextcloud/opt-out-apps

nextcloud · Nov 15, 2023 · 1c2d513 · 1c2d513
2 parents 2f257ee + 19dfe0b
commit 1c2d513
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/js/script.js b/js/script.js
@@ -324,8 +324,8 @@ function updateMonitoringUrl(event) {
 		if ($param.name === 'format_json' && $param.checked) {
 			url.searchParams.set('format', 'json')
 		}
-		if ($param.name === 'skip_apps' && $param.checked) {
-			url.searchParams.set('skipApps', 'true')
+		if ($param.name === 'skip_apps' && !$param.checked) {
+			url.searchParams.set('skipApps', 'false')
 		}
 		if ($param.name === 'skip_update' && !$param.checked) {
 			url.searchParams.set('skipUpdate', 'false')

diff --git a/lib/Controller/ApiController.php b/lib/Controller/ApiController.php
@@ -113,7 +113,7 @@ private function checkAuthorized(): bool {
 	 * @PublicPage
 	 * @BruteForceProtection(action=serverinfo)
 	 */
-	public function info(bool $skipApps = false, bool $skipUpdate = true): DataResponse {
+	public function info(bool $skipApps = true, bool $skipUpdate = true): DataResponse {
 		if (!$this->checkAuthorized()) {
 			$response = new DataResponse(['message' => 'Unauthorized']);
 			$response->throttle();

diff --git a/templates/settings-admin.php b/templates/settings-admin.php
@@ -425,7 +425,7 @@ function FormatMegabytes(int $byte): string {
 						<label for="format_json"><?php p($l->t('Output in JSON')) ?></label>
 					</div>
 					<div class="monitoring-url-param">
-						<input type="checkbox" class="update-monitoring-endpoint-url" name="skip_apps" id="skip_apps">
+						<input type="checkbox" class="update-monitoring-endpoint-url" name="skip_apps" id="skip_apps" checked>
 						<label for="skip_apps"><?php p($l->t('Skip app updates (including app updates will send an external request to the app store)')) ?></label>
 					</div>
 					<div class="monitoring-url-param">