fix: pick up possible exceptions from create method

Signed-off-by: Denis Golovin <[email protected]>
benoitf · Aug 16, 2023 · 0cabb65 · 0cabb65
1 parent 9bf508e
commit 0cabb65
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/packages/main/src/plugin/provider-registry.ts b/packages/main/src/plugin/provider-registry.ts
@@ -706,20 +706,21 @@ export class ProviderRegistry {
     const provider = this.getMatchingProvider(internalProviderId);
     const telemetryData: {
       providerId: string;
-      error?: Error;
+      error?: unknown;
       name: string;
     } = {
       providerId: provider.id,
       name: provider.name,
     };
     try {
       if (!provider.containerProviderConnectionFactory) {
-        const error = new Error('The provider does not support container connection creation');
-        telemetryData.error = error;
-        throw error;
+        throw new Error('The provider does not support container connection creation');
       }
       // create a logger
       return provider.containerProviderConnectionFactory.create(params, logHandler, token);
+    } catch (err) {
+      telemetryData.error = err;
+      throw err;
     } finally {
       await this.telemetryService.track('createProviderConnection', telemetryData);
     }