catch expiration bug (#676)

* fix: add logs and catch error

* ci.yaml: remove unnecessary steps in the CI workflow

service.ts: remove unnecessary try-catch block for checking key expiration

.github/workflows/ci.yaml

@@ -35,9 +35,7 @@ jobs:
            options: --health-cmd="mysqladmin ping" --health-interval=10s --health-timeout=5s --health-retries=3
 
     steps:
-
       - uses: actions/checkout@v4
-
       - uses: actions/setup-go@v4
         with:
           go-version: '^1.20'

apps/api/src/pkg/keys/service.ts

@@ -160,15 +160,7 @@ export class KeyService {
     /**
      * Expiration
      */
-    try {
-      if (data.key.expires && data.key.expires.getTime() < Date.now()) {
-        return result.success({ valid: false, code: "NOT_FOUND" });
-      }
-    } catch (e) {
-      this.logger.error("Failed to check key expiration", {
-        error: (e as Error).message,
-        key: data.key.id,
-      });
+    if (data.key.expires && data.key.expires.getTime() < Date.now()) {
       return result.success({ valid: false, code: "NOT_FOUND" });
     }