From f59111025b762d8067015e6d14d6295f9cde36b5 Mon Sep 17 00:00:00 2001
From: Donald Zou <donaldzou@live.hk>
Date: Mon, 11 Aug 2025 14:35:30 +0800
Subject: [PATCH] Added client profile update

---
 src/dashboard.py                              |  8 +++
 src/modules/DashboardClients.py               | 16 +++++
 src/modules/DashboardConfig.py                |  3 +
 src/static/app/package.json                   |  1 +
 .../clientSettingComponents/oidcSettings.vue  | 13 ++--
 .../clientComponents/clientSettings.vue       | 26 +++++++-
 .../clientComponents/clientViewer.vue         | 64 +++++++++++++++----
 src/static/app/src/views/settings.vue         |  2 -
 8 files changed, 109 insertions(+), 24 deletions(-)

diff --git a/src/dashboard.py b/src/dashboard.py
index 838b0b9..06e4c70 100644
--- a/src/dashboard.py
+++ b/src/dashboard.py
@@ -1325,6 +1325,14 @@ def API_Clients_GeneratePasswordResetLink():
         return ResponseObject(data=token)
     return ResponseObject(False, "Failed to generate link")
 
+@app.post(f'{APP_PREFIX}/api/clients/updateProfileName')
+def API_Clients_UpdateProfile():
+    data = request.get_json()
+    clientId = data.get("ClientID")
+    if not clientId:
+        return ResponseObject(False, "Please provide ClientID")
+    value = data.get('Name')
+    return ResponseObject(status=DashboardClients.UpdateClientProfile(clientId, value))
 '''
 Index Page
 '''
diff --git a/src/modules/DashboardClients.py b/src/modules/DashboardClients.py
index 503e2d5..bb1ab06 100644
--- a/src/modules/DashboardClients.py
+++ b/src/modules/DashboardClients.py
@@ -331,6 +331,22 @@ class DashboardClients:
             return False, "Signed up failed."
         return True, None
     
+    def UpdateClientProfile(self, ClientID, Name):
+        try:
+            with self.engine.begin() as conn:
+                conn.execute(
+                    self.dashboardClientsInfoTable.update().values({
+                        "Name": Name
+                    }).where(
+                        self.dashboardClientsInfoTable.c.ClientID == ClientID
+                    )
+                )
+            self.logger.log(Message=f"User {ClientID} updated name to {Name}")
+        except Exception as e:
+            self.logger.log(Status="false", Message=f"User {ClientID} updated name to {Name} failed")
+            return False
+        return True
+        
     '''
     For WGDashboard Admin to Manage Clients
     '''
diff --git a/src/modules/DashboardConfig.py b/src/modules/DashboardConfig.py
index eb4c174..798638d 100644
--- a/src/modules/DashboardConfig.py
+++ b/src/modules/DashboardConfig.py
@@ -80,6 +80,9 @@ class DashboardConfig:
                 "admin_enable": "false",
                 "client_enable": "false"
             },
+            "Clients": {
+                "allow_local_sign_up": "true"
+            },
             "WireGuardConfiguration": {
                 "autostart": ""
             }
diff --git a/src/static/app/package.json b/src/static/app/package.json
index c91f53a..28d2ae1 100644
--- a/src/static/app/package.json
+++ b/src/static/app/package.json
@@ -3,6 +3,7 @@
   "version": "4.3",
   "private": true,
   "type": "module",
+  "module": "es2022",
   "scripts": {
     "dev": "vite",
     "build": "vite build",
diff --git a/src/static/app/src/components/clientComponents/clientSettingComponents/oidcSettings.vue b/src/static/app/src/components/clientComponents/clientSettingComponents/oidcSettings.vue
index a38b0f0..23f6687 100644
--- a/src/static/app/src/components/clientComponents/clientSettingComponents/oidcSettings.vue
+++ b/src/static/app/src/components/clientComponents/clientSettingComponents/oidcSettings.vue
@@ -1,28 +1,27 @@
 <script setup lang="ts">
 import {ref, watch} from "vue"
 import LocaleText from "@/components/text/localeText.vue";
-const props = defineProps(['mode'])
 import { fetchGet } from "@/utilities/fetch.js"
 import { DashboardConfigurationStore } from "@/stores/DashboardConfigurationStore"
 
+const props = defineProps(['mode'])
 const dashboardConfigurationStore = DashboardConfigurationStore()
 const oidcStatus = ref(false)
 const oidcStatusLoading = ref(false)
 
 const getStatus = async () => {
 	await fetchGet("/api/oidc/status", {
-		mode: "Client"
+		mode: props.mode
 	}, (res) => {
 		oidcStatus.value = res.data
 		oidcStatusLoading.value = false
 	})
 }
-
 await getStatus()
 const toggle = async () => {
 	oidcStatusLoading.value = true
 	await fetchGet('/api/oidc/toggle', {
-		mode: "Client"
+		mode: props.mode
 	}, (res) => {
 		if (!res.status){
 			oidcStatus.value = !oidcStatus.value
@@ -31,7 +30,6 @@ const toggle = async () => {
 		oidcStatusLoading.value = false
 	})
 }
-
 </script>
 
 <template>
@@ -54,12 +52,9 @@ const toggle = async () => {
 	<div>
 		<div class="alert alert-dark rounded-3 mb-0">
 			<LocaleText t="Due to security reason, in order to edit OIDC configuration, you will need to edit "></LocaleText>
-			<code>wg-dashboard-oidc-providers.json</code> <LocaleText t="directly. If you updated the file, click the update button below to reload the configuration."></LocaleText>
+			<code>wg-dashboard-oidc-providers.json</code> <LocaleText t="directly, then restart WGDashboard to apply the latest settings."></LocaleText>
 		</div>
 	</div>
-	<button class="rounded-3 btn bg-success-subtle text-success-emphasis border-success-subtle">
-		<i class="bi bi-arrow-clockwise me-2"></i><LocaleText t="Refresh"></LocaleText>
-	</button>
 </div>
 </template>
 
diff --git a/src/static/app/src/components/clientComponents/clientSettings.vue b/src/static/app/src/components/clientComponents/clientSettings.vue
index 645abb1..5e8d985 100644
--- a/src/static/app/src/components/clientComponents/clientSettings.vue
+++ b/src/static/app/src/components/clientComponents/clientSettings.vue
@@ -1,8 +1,14 @@
 <script setup lang="ts">
+import { ref, reactive } from "vue"
 import LocaleText from "@/components/text/localeText.vue";
 import OidcSettings from "@/components/clientComponents/clientSettingComponents/oidcSettings.vue";
 const emits = defineEmits(['close'])
-
+import { DashboardConfigurationStore } from "@/stores/DashboardConfigurationStore"
+const dashboardConfigurationStore = DashboardConfigurationStore()
+const loading = ref(false)
+const values = reactive({
+	allow_local_sign_up: dashboardConfigurationStore.Configuration.Clients.allow_local_sign_up
+})
 </script>
 
 <template>
@@ -17,7 +23,23 @@ const emits = defineEmits(['close'])
 		<div class="card-body px-4">
 			<div class="py-2">
 				<OidcSettings mode="Client"></OidcSettings>
-				<hr>
+<!--				<hr>-->
+<!--				<div>-->
+<!--					<div class="d-flex">-->
+<!--						<h6 class="mb-0">-->
+<!--							<LocaleText t="Allow Local Accounts Sign Up"></LocaleText>-->
+<!--						</h6>-->
+<!--						<div class="form-check form-switch ms-auto">-->
+<!--							<label class="form-check-label" for="allow_local_sign_up">-->
+<!--								<LocaleText :t="values.allow_local_sign_up ? 'Enabled':'Disabled'"></LocaleText>-->
+<!--							</label>-->
+<!--							<input-->
+<!--								:disabled="loading"-->
+<!--								v-model="values.allow_local_sign_up"-->
+<!--								class="form-check-input" type="checkbox" role="switch" id="allow_local_sign_up">-->
+<!--						</div>-->
+<!--					</div>-->
+<!--				</div>-->
 			</div>
 		</div>
 	</div>
diff --git a/src/static/app/src/components/clientComponents/clientViewer.vue b/src/static/app/src/components/clientComponents/clientViewer.vue
index 08ee4b6..b30a3b8 100644
--- a/src/static/app/src/components/clientComponents/clientViewer.vue
+++ b/src/static/app/src/components/clientComponents/clientViewer.vue
@@ -1,20 +1,22 @@
 <script setup lang="ts" async>
 import {useRoute} from "vue-router";
-import { fetchGet } from "@/utilities/fetch.js"
+import { fetchGet, fetchPost } from "@/utilities/fetch.js"
 
 
 import {DashboardClientAssignmentStore} from "@/stores/DashboardClientAssignmentStore.js";
-import {computed, ref, watch} from "vue";
+import { DashboardConfigurationStore } from "@/stores/DashboardConfigurationStore.js"
+
+import {computed, reactive, ref, watch} from "vue";
 import LocaleText from "@/components/text/localeText.vue";
 import ClientAssignedPeers from "@/components/clientComponents/clientAssignedPeers.vue";
 import ClientResetPassword from "@/components/clientComponents/clientResetPassword.vue";
 const assignmentStore = DashboardClientAssignmentStore()
-const route = useRoute()
+const dashboardConfigurationStore = DashboardConfigurationStore()
 
+const route = useRoute()
 const client = computed(() => {
 	return assignmentStore.getClientById(route.params.id)
 })
-
 const clientAssignedPeers = ref({})
 const getAssignedPeers = async () => {
 	await fetchGet('/api/clients/assignedPeers', {
@@ -24,12 +26,36 @@ const getAssignedPeers = async () => {
 	})
 }
 
+const clientProfile = reactive({
+	Name: client.value.Name
+})
+
 if (client.value){
 	watch(() => client.value.ClientID, async () => {
+		clientProfile.Name = client.value.Name;
 		await getAssignedPeers()
 	})
 	await getAssignedPeers()
 }
+
+const updatingProfile = ref(false)
+const updateProfile = async () => {
+	updatingProfile.value = true
+	await fetchPost("/api/clients/updateProfileName", {
+		ClientID: client.value.ClientID,
+		Name: clientProfile.Name
+	}, (res) => {
+		if (res.status){
+			client.value.Name = clientProfile.Name;
+			dashboardConfigurationStore.newMessage("Server", "Client name update success", "success")
+		}else{
+			clientProfile.Name = client.value.Name;
+			dashboardConfigurationStore.newMessage("Server", "Client name update failed", "danger")
+		}
+		updatingProfile.value = false
+	})
+}
+
 </script>
 
 <template>
@@ -46,13 +72,29 @@ if (client.value){
 			<h1>
 				{{ client.Email }}
 			</h1>
-			<div class="d-flex align-items-center">
-				<small class="text-muted">
-					<LocaleText t="Client ID"></LocaleText>
-				</small>
-				<small class="ms-auto">
-					<samp>{{ client.ClientID }}</samp>
-				</small>
+			<div class="d-flex flex-column gap-2">
+				<div class="d-flex align-items-center">
+					<small class="text-muted">
+						<LocaleText t="Client ID"></LocaleText>
+					</small>
+					<small class="ms-auto">
+						<samp>{{ client.ClientID }}</samp>
+					</small>
+				</div>
+				<div class="d-flex align-items-center gap-2">
+					<small class="text-muted">
+						<LocaleText t="Client Name"></LocaleText>
+					</small>
+					<input class="form-control form-control-sm rounded-3 ms-auto"
+						   style="width: 300px"
+						   type="text" v-model="clientProfile.Name">
+					<button
+						@click="updateProfile()"
+						aria-label="Save Client Name"
+						class="btn btn-sm rounded-3 bg-success-subtle border-success-subtle text-success-emphasis">
+						<i class="bi bi-save-fill"></i>
+					</button>
+				</div>
 			</div>
 		</div>
 		<div style="flex: 1 0 0; overflow-y: scroll;">
diff --git a/src/static/app/src/views/settings.vue b/src/static/app/src/views/settings.vue
index 45ba9e6..dc09002 100644
--- a/src/static/app/src/views/settings.vue
+++ b/src/static/app/src/views/settings.vue
@@ -1,7 +1,6 @@
 <script>
 import {DashboardConfigurationStore} from "@/stores/DashboardConfigurationStore.js";
 import PeersDefaultSettingsInput from "@/components/settingsComponent/peersDefaultSettingsInput.vue";
-import {ipV46RegexCheck} from "@/utilities/ipCheck.js";
 import AccountSettingsInputUsername from "@/components/settingsComponent/accountSettingsInputUsername.vue";
 import AccountSettingsInputPassword from "@/components/settingsComponent/accountSettingsInputPassword.vue";
 import DashboardSettingsInputWireguardConfigurationPath
@@ -20,7 +19,6 @@ import DashboardEmailSettings from "@/components/settingsComponent/dashboardEmai
 
 export default {
 	name: "settings",
-	methods: {ipV46RegexCheck},
 	components: {
 		DashboardEmailSettings,
 		DashboardSettingsWireguardConfigurationAutostart,