From 0948b628a410a3fcae0d76afa4bd787e406a00f4 Mon Sep 17 00:00:00 2001
From: Bernd Storath <999999bst@gmail.com>
Date: Sun, 25 Aug 2024 20:54:52 +0200
Subject: [PATCH] fix regressions

fix missing expire date in client create dialog
fix wrong type rules
fix wrong api endpoint
---
 src/components/Clients/CreateDialog.vue | 18 ++++++++++++++++++
 src/server/utils/types.ts               |  7 +++----
 src/stores/modal.ts                     |  4 ++--
 src/utils/api.ts                        |  2 +-
 4 files changed, 24 insertions(+), 7 deletions(-)

diff --git a/src/components/Clients/CreateDialog.vue b/src/components/Clients/CreateDialog.vue
index 5488ee05..81affb69 100644
--- a/src/components/Clients/CreateDialog.vue
+++ b/src/components/Clients/CreateDialog.vue
@@ -71,6 +71,23 @@
                   />
                 </p>
               </div>
+              <div v-show="globalStore.enableExpireTime" class="mt-2">
+                <p class="text-sm text-gray-500">
+                  <label
+                    class="block text-gray-900 dark:text-neutral-200 text-sm font-bold mb-2"
+                    for="expireDate"
+                  >
+                    {{ $t('ExpireDate') }}
+                  </label>
+                  <input
+                    v-model.trim="modalStore.clientExpireDate"
+                    class="rounded p-2 border-2 dark:bg-neutral-700 dark:text-neutral-200 border-gray-100 dark:border-neutral-600 focus:border-gray-200 focus:dark:border-neutral-500 dark:placeholder:text-neutral-400 outline-none w-full"
+                    type="date"
+                    :placeholder="$t('ExpireDate')"
+                    name="expireDate"
+                  />
+                </p>
+              </div>
             </div>
           </div>
         </div>
@@ -110,4 +127,5 @@
 
 <script setup lang="ts">
 const modalStore = useModalStore();
+const globalStore = useGlobalStore();
 </script>
diff --git a/src/server/utils/types.ts b/src/server/utils/types.ts
index bb97abcc..3b19c4a2 100644
--- a/src/server/utils/types.ts
+++ b/src/server/utils/types.ts
@@ -30,14 +30,13 @@ const password = z
   .string({ message: 'Password must be a valid string' })
   .pipe(safeStringRefine);
 
-const remember = z
-  .boolean({ message: 'Remember must be a valid boolean' })
-  .optional();
+const remember = z.boolean({ message: 'Remember must be a valid boolean' });
 
 const expireDate = z
   .string({ message: 'expiredDate must be a valid string' })
   .min(1, 'expiredDate must be at least 1 Character')
-  .pipe(safeStringRefine);
+  .pipe(safeStringRefine)
+  .nullable();
 
 export const clientIdType = z.object(
   {
diff --git a/src/stores/modal.ts b/src/stores/modal.ts
index 99e5c3ca..c44fa7ab 100644
--- a/src/stores/modal.ts
+++ b/src/stores/modal.ts
@@ -5,12 +5,12 @@ export const useModalStore = defineStore('Modal', () => {
   const clientDelete = ref<null | WGClient>(null);
   const clientCreate = ref<null | boolean>(null);
   const clientCreateName = ref<string>('');
-  const clientExpireDate = ref<string | null>(null);
+  const clientExpireDate = ref<string>('');
   const qrcode = ref<null | string>(null);
 
   function createClient() {
     const name = clientCreateName.value;
-    const expireDate = clientExpireDate.value;
+    const expireDate = clientExpireDate.value || null;
     if (!name) return;
 
     api
diff --git a/src/utils/api.ts b/src/utils/api.ts
index a3a0f133..15553875 100644
--- a/src/utils/api.ts
+++ b/src/utils/api.ts
@@ -93,7 +93,7 @@ class API {
   }
 
   async showOneTimeLink({ clientId }: { clientId: string }) {
-    return $fetch(`/api/wireguard/${clientId}/:clientId/generateOneTimeLink`, {
+    return $fetch(`/api/wireguard/client/${clientId}/generateOneTimeLink`, {
       method: 'post',
     });
   }