@sphereon/ssi-sdk.kms-rest-client 0.34.1-feature.IDK.11.296 → 0.34.1-feature.IDK.11.298

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@sphereon/ssi-sdk.kms-rest-client",
   "description": "contains the client side to call REST endpoints of a KMS server",
-  "version": "0.34.1-feature.IDK.11.296+664623d2",
+  "version": "0.34.1-feature.IDK.11.298+9b01cf08",
   "source": "src/index.ts",
   "type": "module",
   "main": "./dist/index.cjs",
@@ -28,7 +28,7 @@
     "generate-plugin-schema": "tsx ../../packages/dev/bin/sphereon.js dev generate-plugin-schema"
   },
   "dependencies": {
-    "@sphereon/ssi-types": "0.34.1-feature.IDK.11.296+664623d2",
+    "@sphereon/ssi-types": "0.34.1-feature.IDK.11.298+9b01cf08",
     "cross-fetch": "^3.1.8"
   },
   "devDependencies": {
@@ -56,5 +56,5 @@
     "KMS",
     "REST"
   ],
-  "gitHead": "664623d271437ff969c1d3cb74cd1a37ee25e492"
+  "gitHead": "9b01cf08191cb9165249a49b043d1675d82826a8"
 }

package/src/agent/KmsRestClient.ts

@@ -116,14 +116,10 @@ export class KmsRestClient implements IAgentPlugin {
     logger.debug(`get resolver response: ${response}`)
 
     if (!response.ok) {
-      return Promise.reject(await response.json())
+      return this.handleHttpError(url, response)
     }
 
-    try {
-      return ResolverFromJSONTyped(await response.json(), false)
-    } catch (error) {
-      return Promise.reject(Error(`request to ${url} returned ${error}`))
-    }
+    return ResolverFromJSONTyped(await response.json(), false)
   }
 
   /** {@inheritDoc IKmsRestClient.kmsListResolvers} */
@@ -137,14 +133,10 @@ export class KmsRestClient implements IAgentPlugin {
     logger.debug(`list resolvers response: ${response}`)
 
     if (!response.ok) {
-      return Promise.reject(await response.json())
+      return this.handleHttpError(url, response)
     }
 
-    try {
-      return ListResolversResponseFromJSONTyped(await response.json(), false)
-    } catch (error) {
-      return Promise.reject(Error(`request to ${url} returned ${error}`))
-    }
+    return ListResolversResponseFromJSONTyped(await response.json(), false)
   }
 
   /** {@inheritDoc IKmsRestClient.kmsResolveKey} */
@@ -166,14 +158,10 @@ export class KmsRestClient implements IAgentPlugin {
     logger.debug(`resolve key response: ${response}`)
 
     if (!response.ok) {
-      return Promise.reject(await response.json())
+      return this.handleHttpError(url, response)
     }
 
-    try {
-      return ResolvedKeyInfoFromJSONTyped(await response.json(), false)
-    } catch (error) {
-      return Promise.reject(Error(`request to ${url} returned ${error}`))
-    }
+    return ResolvedKeyInfoFromJSONTyped(await response.json(), false)
   }
 
   /** {@inheritDoc IKmsRestClient.kmsCreateRawSignature} */
@@ -193,14 +181,10 @@ export class KmsRestClient implements IAgentPlugin {
     logger.debug(`create raw signature response: ${response}`)
 
     if (!response.ok) {
-      return Promise.reject(await response.json())
+      return this.handleHttpError(url, response)
     }
 
-    try {
-      return CreateRawSignatureResponseFromJSONTyped(await response.json(), false)
-    } catch (error) {
-      return Promise.reject(Error(`request to ${url} returned ${error}`))
-    }
+    return CreateRawSignatureResponseFromJSONTyped(await response.json(), false)
   }
 
   /** {@inheritDoc IKmsRestClient.kmsIsValidRawSignature} */
@@ -221,14 +205,10 @@ export class KmsRestClient implements IAgentPlugin {
     logger.debug(`verify raw signature response: ${response}`)
 
     if (!response.ok) {
-      return Promise.reject(await response.json())
+      return this.handleHttpError(url, response)
     }
 
-    try {
-      return VerifyRawSignatureResponseFromJSONTyped(await response.json(), false)
-    } catch (error) {
-      return Promise.reject(Error(`request to ${url} returned ${error}`))
-    }
+    return VerifyRawSignatureResponseFromJSONTyped(await response.json(), false)
   }
 
   /** {@inheritDoc IKmsRestClient.kmsGetKey} */
@@ -242,14 +222,10 @@ export class KmsRestClient implements IAgentPlugin {
     logger.debug(`get key response: ${response}`)
 
     if (!response.ok) {
-      return Promise.reject(await response.json())
+      return this.handleHttpError(url, response)
     }
 
-    try {
-      return GetKeyResponseFromJSONTyped(await response.json(), false)
-    } catch (error) {
-      return Promise.reject(Error(`request to ${url} returned ${error}`))
-    }
+    return GetKeyResponseFromJSONTyped(await response.json(), false)
   }
 
   /** {@inheritDoc IKmsRestClient.kmsListKeys} */
@@ -262,14 +238,10 @@ export class KmsRestClient implements IAgentPlugin {
     logger.debug(`list keys response: ${response}`)
 
     if (!response.ok) {
-      return Promise.reject(await response.json())
+      return this.handleHttpError(url.toString(), response)
     }
 
-    try {
-      return ListKeysResponseFromJSONTyped(await response.json(), false)
-    } catch (error) {
-      return Promise.reject(Error(`request to ${url} returned ${error}`))
-    }
+    return ListKeysResponseFromJSONTyped(await response.json(), false)
   }
 
   /** {@inheritDoc IKmsRestClient.kmsStoreKey} */
@@ -289,14 +261,10 @@ export class KmsRestClient implements IAgentPlugin {
     logger.debug(`store key response: ${response}`)
 
     if (!response.ok) {
-      return Promise.reject(await response.json())
+      return this.handleHttpError(url, response)
     }
 
-    try {
-      return StoreKeyResponseFromJSONTyped(await response.json(), false)
-    } catch (error) {
-      return Promise.reject(Error(`request to ${url} returned ${error}`))
-    }
+    return StoreKeyResponseFromJSONTyped(await response.json(), false)
   }
 
   /** {@inheritDoc IKmsRestClient.kmsGenerateKey} */
@@ -318,14 +286,10 @@ export class KmsRestClient implements IAgentPlugin {
     logger.debug(`generate key response: ${response}`)
 
     if (!response.ok) {
-      return Promise.reject(await response.json())
+      return this.handleHttpError(url, response)
     }
 
-    try {
-      return GenerateKeyResponseFromJSONTyped(await response.json(), false)
-    } catch (error) {
-      return Promise.reject(Error(`request to ${url} returned ${error}`))
-    }
+    return GenerateKeyResponseFromJSONTyped(await response.json(), false)
   }
 
   /** {@inheritDoc IKmsRestClient.kmsDeleteKey} */
@@ -336,9 +300,10 @@ export class KmsRestClient implements IAgentPlugin {
     const response = await fetch(url, {
       method: 'DELETE',
     })
+    logger.debug(`delete key response: ${response}`)
 
     if (!response.ok) {
-      return Promise.reject(await response.json())
+      return this.handleHttpError(url, response)
     }
 
     return true
@@ -355,14 +320,10 @@ export class KmsRestClient implements IAgentPlugin {
     logger.debug(`get provider response: ${response}`)
 
     if (!response.ok) {
-      return Promise.reject(await response.json())
+      return this.handleHttpError(url, response)
     }
 
-    try {
-      return KeyProviderResponseFromJSONTyped(await response.json(), false)
-    } catch (error) {
-      return Promise.reject(Error(`request to ${url} returned ${error}`))
-    }
+    return KeyProviderResponseFromJSONTyped(await response.json(), false)
   }
 
   /** {@inheritDoc IKmsRestClient.kmsListKeyProviders} */
@@ -376,14 +337,10 @@ export class KmsRestClient implements IAgentPlugin {
     logger.debug(`list providers response: ${response}`)
 
     if (!response.ok) {
-      return Promise.reject(await response.json())
+      return this.handleHttpError(url, response)
     }
 
-    try {
-      return ListKeyProvidersResponseFromJSONTyped(await response.json(), false)
-    } catch (error) {
-      return Promise.reject(Error(`request to ${url} returned ${error}`))
-    }
+    return ListKeyProvidersResponseFromJSONTyped(await response.json(), false)
   }
 
   /** {@inheritDoc IKmsRestClient.kmsProviderListKeys} */
@@ -397,14 +354,10 @@ export class KmsRestClient implements IAgentPlugin {
     logger.debug(`list provider keys response: ${response}`)
 
     if (!response.ok) {
-      return Promise.reject(await response.json())
+      return this.handleHttpError(url, response)
     }
 
-    try {
-      return ListKeysResponseFromJSONTyped(await response.json(), false)
-    } catch (error) {
-      return Promise.reject(Error(`request to ${url} returned ${error}`))
-    }
+    return ListKeysResponseFromJSONTyped(await response.json(), false)
   }
 
   /** {@inheritDoc IKmsRestClient.kmsProviderStoreKey} */
@@ -424,14 +377,10 @@ export class KmsRestClient implements IAgentPlugin {
     logger.debug(`provider store key response: ${response}`)
 
     if (!response.ok) {
-      return Promise.reject(await response.json())
+      return this.handleHttpError(url, response)
     }
 
-    try {
-      return StoreKeyResponseFromJSONTyped(await response.json(), false)
-    } catch (error) {
-      return Promise.reject(Error(`request to ${url} returned ${error}`))
-    }
+    return StoreKeyResponseFromJSONTyped(await response.json(), false)
   }
 
   /** {@inheritDoc IKmsRestClient.kmsProviderGenerateKey} */
@@ -452,14 +401,10 @@ export class KmsRestClient implements IAgentPlugin {
     logger.debug(`provider generate key response: ${response}`)
 
     if (!response.ok) {
-      return Promise.reject(await response.json())
+      return this.handleHttpError(url, response)
     }
 
-    try {
-      return GenerateKeyResponseFromJSONTyped(await response.json(), false)
-    } catch (error) {
-      return Promise.reject(Error(`request to ${url} returned ${error}`))
-    }
+    return GenerateKeyResponseFromJSONTyped(await response.json(), false)
   }
 
   /** {@inheritDoc IKmsRestClient.kmsProviderGetKey} */
@@ -473,14 +418,10 @@ export class KmsRestClient implements IAgentPlugin {
     logger.debug(`get provider key response: ${response}`)
 
     if (!response.ok) {
-      return Promise.reject(await response.json())
+      return this.handleHttpError(url, response)
     }
 
-    try {
-      return GetKeyResponseFromJSONTyped(await response.json(), false)
-    } catch (error) {
-      return Promise.reject(Error(`request to ${url} returned ${error}`))
-    }
+    return GetKeyResponseFromJSONTyped(await response.json(), false)
   }
 
   /** {@inheritDoc IKmsRestClient.kmsProviderDeleteKey} */
@@ -491,9 +432,10 @@ export class KmsRestClient implements IAgentPlugin {
     const response = await fetch(url, {
       method: 'DELETE',
     })
+    logger.debug(`delete key response: ${response}`)
 
     if (!response.ok) {
-      return Promise.reject(await response.json())
+      return this.handleHttpError(url, response)
     }
 
     return true
@@ -533,4 +475,14 @@ export class KmsRestClient implements IAgentPlugin {
 
     return url
   }
+
+  private async handleHttpError(url: string, response: Response): Promise<never> {
+    const contentType = response.headers.get("content-type")
+    const isJson = contentType?.includes("application/json")
+    const responseBody = isJson
+      ? JSON.stringify(await response.json())
+      : await response.text()
+
+    return Promise.reject(Error(`request to ${url} returned http error ${response.status} - ${responseBody}`))
+  }
 }