improve code

.gitignore

@@ -4,5 +4,3 @@
 /src/node_modules
 .DS_Store
 *.swp
-# lowdb data file
-db.json

src/server/api/cnf/[oneTimeLink].ts

@@ -6,7 +6,6 @@ export default defineEventHandler(async (event) => {
       statusMessage: 'Invalid state',
     });
   }
-  // TODO: validate with zod
   const { oneTimeLink } = await getValidatedRouterParams(
     event,
     validateZod(oneTimeLinkType)

src/server/api/release.get.ts

@@ -1,11 +1,5 @@
 export default defineEventHandler(async () => {
   const system = await Database.getSystem();
-  if (!system)
-    throw createError({
-      statusCode: 500,
-      statusMessage: 'Invalid',
-    });
-
   const latestRelease = await fetchLatestRelease();
   return {
     currentRelease: system.release,

src/server/api/session.post.ts

@@ -23,11 +23,6 @@ export default defineEventHandler(async (event) => {
   }
 
   const system = await Database.getSystem();
-  if (!system)
-    throw createError({
-      statusCode: 500,
-      statusMessage: 'Invalid',
-    });
 
   const conf: SessionConfig = system.sessionConfig;
 

src/server/middleware/session.ts

@@ -2,12 +2,12 @@ export default defineEventHandler(async (event) => {
   const url = getRequestURL(event);
   if (
     !url.pathname.startsWith('/api/') ||
+    // TODO: only allowed on onboarding!
     url.pathname === '/api/account/new' ||
     url.pathname === '/api/session' ||
     url.pathname === '/api/lang' ||
     url.pathname === '/api/release' ||
-    url.pathname === '/api/ui-chart-type' ||
-    url.pathname === '/api/ui-traffic-stats'
+    url.pathname === '/api/features'
   ) {
     return;
   }

src/services/database/lowdb.ts

@@ -22,7 +22,6 @@ export default class LowDB extends DatabaseProvider {
 
   // is this really needed?
   private async __init() {
-    // TODO: assume path to db file
     const dbFilePath = join('/etc/wireguard', 'db.json');
     this.#db = await JSONFilePreset(dbFilePath, DEFAULT_DATABASE);
   }
@@ -80,6 +79,7 @@ export default class LowDB extends DatabaseProvider {
       throw new DatabaseError(DatabaseError.ERROR_PASSWORD_REQ);
     }
 
+    // TODO: multiple names are no problem
     const isUserExist = this.#db.data.users.find(
       (user) => user.username === username
     );

src/services/database/migrations/1.ts

@@ -23,7 +23,7 @@ export async function run1(db: Low<Database>) {
       userConfig: {
         mtu: 1420,
         persistentKeepalive: 0,
-        // TODO: assume handle CIDR to compute next ip in WireGuard
+        // TODO: handle CIDR to compute next ip in WireGuard
         //addressRange: '10.8.0.0/24',
         addressRange: addressRange,
         defaultDns: ['1.1.1.1'],