Skip to content

Commit fae245a

Browse files
committed
Let extension to work normally in case of telemetry init failure
This PR fixes #2174. Signed-off-by: Denis Golovin dgolovin@redhat.com
1 parent f22a6d3 commit fae245a

File tree

1 file changed

+9
-7
lines changed

1 file changed

+9
-7
lines changed

src/telemetry.ts

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -17,16 +17,18 @@ export function createTrackingEvent(name: string, properties: any = {}): Telemet
1717
}
1818

1919
export async function startTelemetry(context: ExtensionContext): Promise<void> {
20-
const redHatService = await getRedHatService(context);
21-
telemetryService = await redHatService.getTelemetryService();
22-
return telemetryService.sendStartupEvent();
20+
try {
21+
const redHatService = await getRedHatService(context);
22+
telemetryService = await redHatService.getTelemetryService();
23+
} catch(error) {
24+
// eslint-disable-next-line no-console
25+
console.log(`${error}`);
26+
}
27+
return telemetryService?.sendStartupEvent();
2328
}
2429

2530
export default async function sendTelemetry(actionName: string, properties?: any): Promise<void> {
26-
if (!telemetryService) {
27-
throw Error('Telemetry service has not been started yet!');
28-
}
29-
return telemetryService.send(createTrackingEvent(actionName, properties));
31+
return telemetryService?.send(createTrackingEvent(actionName, properties));
3032
}
3133

3234
export interface CommonCommandProps {

0 commit comments

Comments
 (0)