@sd-jwt/sd-jwt-vc 0.9.2-next.6 → 0.9.2-next.8

package/dist/index.js

@@ -179,14 +179,9 @@ var SDJwtVcInstance = class _SDJwtVcInstance extends import_core.SDJwtInstance {
   fetch(url, integrity) {
     return __async(this, null, function* () {
       var _a;
-      const controller = new AbortController();
-      const timeoutId = setTimeout(
-        () => controller.abort(),
-        (_a = this.userConfig.timeout) != null ? _a : 1e4
-      );
       try {
         const response = yield fetch(url, {
-          signal: controller.signal
+          signal: AbortSignal.timeout((_a = this.userConfig.timeout) != null ? _a : 1e4)
         });
         if (!response.ok) {
           const errorText = yield response.text();
@@ -197,12 +192,10 @@ var SDJwtVcInstance = class _SDJwtVcInstance extends import_core.SDJwtInstance {
         yield this.validateIntegrity(response.clone(), url, integrity);
         return response.json();
       } catch (error) {
-        if (error instanceof DOMException && error.name === "AbortError") {
+        if (error.name === "TimeoutError") {
           throw new Error(`Request to ${url} timed out`);
         }
         throw error;
-      } finally {
-        clearTimeout(timeoutId);
       }
     });
   }

package/dist/index.mjs

@@ -147,14 +147,9 @@ var SDJwtVcInstance = class _SDJwtVcInstance extends SDJwtInstance {
   fetch(url, integrity) {
     return __async(this, null, function* () {
       var _a;
-      const controller = new AbortController();
-      const timeoutId = setTimeout(
-        () => controller.abort(),
-        (_a = this.userConfig.timeout) != null ? _a : 1e4
-      );
       try {
         const response = yield fetch(url, {
-          signal: controller.signal
+          signal: AbortSignal.timeout((_a = this.userConfig.timeout) != null ? _a : 1e4)
         });
         if (!response.ok) {
           const errorText = yield response.text();
@@ -165,12 +160,10 @@ var SDJwtVcInstance = class _SDJwtVcInstance extends SDJwtInstance {
         yield this.validateIntegrity(response.clone(), url, integrity);
         return response.json();
       } catch (error) {
-        if (error instanceof DOMException && error.name === "AbortError") {
+        if (error.name === "TimeoutError") {
           throw new Error(`Request to ${url} timed out`);
         }
         throw error;
-      } finally {
-        clearTimeout(timeoutId);
       }
     });
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sd-jwt/sd-jwt-vc",
-  "version": "0.9.2-next.6+5e64551",
+  "version": "0.9.2-next.8+ed907e3",
   "description": "sd-jwt draft 7 implementation in typescript",
   "main": "dist/index.js",
   "module": "dist/index.mjs",
@@ -39,15 +39,15 @@
   },
   "license": "Apache-2.0",
   "dependencies": {
-    "@sd-jwt/core": "0.9.2-next.6+5e64551",
-    "@sd-jwt/jwt-status-list": "0.9.2-next.6+5e64551",
-    "@sd-jwt/utils": "0.9.2-next.6+5e64551",
+    "@sd-jwt/core": "0.9.2-next.8+ed907e3",
+    "@sd-jwt/jwt-status-list": "0.9.2-next.8+ed907e3",
+    "@sd-jwt/utils": "0.9.2-next.8+ed907e3",
     "ajv": "^8.17.1",
     "ajv-formats": "^3.0.1"
   },
   "devDependencies": {
-    "@sd-jwt/crypto-nodejs": "0.9.2-next.6+5e64551",
-    "@sd-jwt/types": "0.9.2-next.6+5e64551",
+    "@sd-jwt/crypto-nodejs": "0.9.2-next.8+ed907e3",
+    "@sd-jwt/types": "0.9.2-next.8+ed907e3",
     "jose": "^5.2.2",
     "msw": "^2.3.5"
   },
@@ -67,5 +67,5 @@
       "esm"
     ]
   },
-  "gitHead": "5e6455158648525d09320dd562422d9f9f33ffaa"
+  "gitHead": "ed907e33c79f140268a03098af1e8c5cc1003ce9"
 }

package/src/sd-jwt-vc-instance.ts

@@ -167,17 +167,10 @@ export class SDJwtVcInstance extends SDJwtInstance<SdJwtVcPayload> {
    * @returns
    */
   private async fetch<T>(url: string, integrity?: string): Promise<T> {
-    const controller = new AbortController();
-    const timeoutId = setTimeout(
-      () => controller.abort(),
-      this.userConfig.timeout ?? 10000,
-    );
-
     try {
       const response = await fetch(url, {
-        signal: controller.signal,
+        signal: AbortSignal.timeout(this.userConfig.timeout ?? 10000),
       });
-
       if (!response.ok) {
         const errorText = await response.text();
         throw new Error(
@@ -187,12 +180,10 @@ export class SDJwtVcInstance extends SDJwtInstance<SdJwtVcPayload> {
       await this.validateIntegrity(response.clone(), url, integrity);
       return response.json() as Promise<T>;
     } catch (error) {
-      if (error instanceof DOMException && error.name === 'AbortError') {
+      if ((error as Error).name === 'TimeoutError') {
         throw new Error(`Request to ${url} timed out`);
       }
       throw error;
-    } finally {
-      clearTimeout(timeoutId);
     }
   }
 

package/src/test/vct.spec.ts

@@ -1,6 +1,6 @@
 import { digest, generateSalt } from '@sd-jwt/crypto-nodejs';
 import type { DisclosureFrame, Signer, Verifier } from '@sd-jwt/types';
-import { describe, test, beforeAll, afterAll } from 'vitest';
+import { describe, test, beforeAll, afterAll, expect } from 'vitest';
 import { SDJwtVcInstance } from '..';
 import type { SdJwtVcPayload } from '../sd-jwt-vc-payload';
 import Crypto from 'node:crypto';
@@ -41,7 +41,7 @@ const restHandlers = [
     };
     return HttpResponse.json(res);
   }),
-  http.get('http://exmaple.com/example', () => {
+  http.get('http://example.com/example', () => {
     const res: TypeMetadataFormat = {
       vct: 'http://example.com/example',
       name: 'ExampleCredentialType',
@@ -53,6 +53,13 @@ const restHandlers = [
     };
     return HttpResponse.json(res);
   }),
+  http.get('http://example.com/timeout', () => {
+    return new Promise((resolve) => {
+      setTimeout(() => {
+        resolve(HttpResponse.json({}));
+      }, 10000);
+    });
+  }),
 ];
 
 //this value could be generated on demand to make it easier when changing the values
@@ -62,7 +69,7 @@ const vctIntegrity =
 const server = setupServer(...restHandlers);
 
 const iss = 'ExampleIssuer';
-const vct = 'http://exmaple.com/example';
+const vct = 'http://example.com/example';
 const iat = new Date().getTime() / 1000;
 
 const { privateKey, publicKey } = Crypto.generateKeyPairSync('ed25519');
@@ -84,6 +91,26 @@ const createSignerVerifier = () => {
 };
 
 describe('App', () => {
+  const { signer, verifier } = createSignerVerifier();
+
+  const sdjwt = new SDJwtVcInstance({
+    signer,
+    signAlg: 'EdDSA',
+    verifier,
+    hasher: digest,
+    hashAlg: 'sha-256',
+    saltGenerator: generateSalt,
+    loadTypeMetadataFormat: true,
+    timeout: 1000,
+  });
+
+  const claims = {
+    firstname: 'John',
+  };
+  const disclosureFrame = {
+    _sd: ['firstname'],
+  };
+
   beforeAll(() => server.listen({ onUnhandledRequest: 'warn' }));
 
   afterAll(() => server.close());
@@ -91,24 +118,6 @@ describe('App', () => {
   afterEach(() => server.resetHandlers());
 
   test('VCT Validation', async () => {
-    const { signer, verifier } = createSignerVerifier();
-    const sdjwt = new SDJwtVcInstance({
-      signer,
-      signAlg: 'EdDSA',
-      verifier,
-      hasher: digest,
-      hashAlg: 'sha-256',
-      saltGenerator: generateSalt,
-      loadTypeMetadataFormat: true,
-    });
-
-    const claims = {
-      firstname: 'John',
-    };
-    const disclosureFrame = {
-      _sd: ['firstname'],
-    };
-
     const expectedPayload: SdJwtVcPayload = {
       iat,
       iss,
@@ -124,5 +133,23 @@ describe('App', () => {
     await sdjwt.verify(encodedSdjwt);
   });
 
+  test('VCT Validation with timeout', async () => {
+    const vct = 'http://example.com/timeout';
+    const expectedPayload: SdJwtVcPayload = {
+      iat,
+      iss,
+      vct,
+      ...claims,
+    };
+    const encodedSdjwt = await sdjwt.issue(
+      expectedPayload,
+      disclosureFrame as unknown as DisclosureFrame<SdJwtVcPayload>,
+    );
+
+    expect(sdjwt.verify(encodedSdjwt)).rejects.toThrowError(
+      `Request to ${vct} timed out`,
+    );
+  });
+
   //TODO: we need tests with an embedded schema, extended and maybe also to test the errors when schema information is not available or the integrity is not valid
 });