From 6544271d3be643a4489aafbeac635061d49273a9 Mon Sep 17 00:00:00 2001
From: Andrey Borysenko <andrey18106x@gmail.com>
Date: Wed, 24 Jan 2024 12:54:05 +0200
Subject: [PATCH] WIP: minor fixes, force host if https enabled in UI

Signed-off-by: Andrey Borysenko <andrey18106x@gmail.com>
---
 lib/DeployActions/DockerActions.php           | 10 +++---
 .../RegisterDaemonConfigModal.vue             | 32 ++++++++++++++++---
 2 files changed, 32 insertions(+), 10 deletions(-)

diff --git a/lib/DeployActions/DockerActions.php b/lib/DeployActions/DockerActions.php
index aa3da9cc..002068d3 100644
--- a/lib/DeployActions/DockerActions.php
+++ b/lib/DeployActions/DockerActions.php
@@ -514,12 +514,15 @@ public function initGuzzleClient(DaemonConfig $daemonConfig): void {
 				],
 			];
 		} elseif ($daemonConfig->getProtocol() === 'https') {
-			$guzzleParams = $this->setupCerts($guzzleParams, $daemonConfig->getDeployConfig());
+			$guzzleParams = $this->setupCerts($guzzleParams);
+		}
+		if (!empty($daemonConfig->getDeployConfig()['haproxy_password'])) {
+			$guzzleParams['auth'] = [self::APP_API_HAPROXY_USER, $daemonConfig->getDeployConfig()['haproxy_password']];
 		}
 		$this->guzzleClient = new Client($guzzleParams);
 	}
 
-	private function setupCerts(array $guzzleParams, array $deployConfig): array {
+	private function setupCerts(array $guzzleParams): array {
 		if (!$this->config->getSystemValueBool('installed', false)) {
 			$certs = \OC::$SERVERROOT . '/resources/config/ca-bundle.crt';
 		} else {
@@ -527,9 +530,6 @@ private function setupCerts(array $guzzleParams, array $deployConfig): array {
 		}
 
 		$guzzleParams['verify'] = $certs;
-		if (!empty($deployConfig['haproxy_password'])) {
-			$guzzleParams['auth'] = [self::APP_API_HAPROXY_USER, $deployConfig['haproxy_password']];
-		}
 		return $guzzleParams;
 	}
 
diff --git a/src/components/DaemonConfig/RegisterDaemonConfigModal.vue b/src/components/DaemonConfig/RegisterDaemonConfigModal.vue
index df1ce014..2ac17051 100644
--- a/src/components/DaemonConfig/RegisterDaemonConfigModal.vue
+++ b/src/components/DaemonConfig/RegisterDaemonConfigModal.vue
@@ -86,16 +86,20 @@
 							</template>
 						</NcButton>
 						<div v-show="deployConfigSettingsOpened" class="deploy-config" :aria-label="t('app_api', 'Deploy config')">
-							<div class="external-label" :aria-label="t('app_api', 'Network')">
+							<div class="external-label"
+								:aria-label="t('app_api', 'Network')">
 								<label for="deploy-config-net">{{ t('app_api', 'Network') }}</label>
 								<NcInputField
 									id="deploy-config-net"
 									:value.sync="deployConfig.net"
+									:disabled="daemonProtocol === 'https'"
 									:placeholder="t('app_api', 'Docker network name')"
 									:aria-label="t('app_api', 'Docker network name')"
-									:helper-text="t('app_api', 'Docker network name')" />
+									:helper-text="networkHelperText" />
 							</div>
-							<div v-if="['http', 'https'].includes(daemonProtocol)" class="external-label" :aria-label="t('app_api', 'HaProxy password')">
+							<div v-if="['http', 'https'].includes(daemonProtocol)"
+								class="external-label"
+								:aria-label="t('app_api', 'HaProxy password')">
 								<label for="deploy-config-haproxy-password">{{ t('app_api', 'HaProxy password') }}</label>
 								<NcInputField
 									id="deploy-config-haproxy-password"
@@ -124,7 +128,7 @@
 					<div class="row">
 						<NcButton
 							type="primary"
-							:disabled="isDaemonNameValid === true && isHaProxyPasswordValid === true"
+							:disabled="canRegister"
 							@click="registerDaemon">
 							{{ t('app_api', 'Register') }}
 							<template #icon>
@@ -248,11 +252,21 @@ export default {
 			if (this.daemonProtocol === 'https') {
 				return this.deployConfig.haproxy_password !== null && this.deployConfig.haproxy_password.length >= 12
 			}
+			// HaProxy password required only for https
 			return true
 		},
 		haProxyPasswordHelperText() {
 			return this.isHaProxyPasswordValid ? t('app_api', 'AppAPI Docker Socket Proxy authentication password') : t('app_api', 'Password must be at least 12 characters long')
 		},
+		networkHelperText() {
+			if (this.httpsEnabled) {
+				return t('app_api', 'With https enabled network is set to host')
+			}
+			return t('app_api', 'Docker network name')
+		},
+		canRegister() {
+			return this.isDaemonNameValid === true || this.isHaProxyPasswordValid === false
+		},
 	},
 	watch: {
 		configurationTab(newConfigurationTab) {
@@ -268,6 +282,14 @@ export default {
 				this.configurationTab = { id: this.configurationTab.id, label: this.configurationTab.label }
 			}
 		},
+		httpsEnabled(newHttpsEnabled) {
+			if (newHttpsEnabled) {
+				this.prevNet = this.deployConfig.net
+				this.deployConfig.net = 'host'
+			} else {
+				this.deployConfig.net = this.prevNet
+			}
+		},
 	},
 	methods: {
 		registerDaemon() {
@@ -352,7 +374,7 @@ export default {
 			}
 		},
 		closeModal() {
-			const customTemplate = DAEMON_TEMPLATES.find(template => template.name === this.configurationTab.id)
+			const customTemplate = DAEMON_TEMPLATES.find(template => template.id === 'custom')
 			this.configurationTab = { id: customTemplate.id, label: customTemplate.displayName }
 			this.$emit('update:show', false)
 		},