Only enable otel if we have a collector URL

This commit is contained in:
David Baker 2023-03-29 16:04:11 +01:00
parent 848e28ef92
commit f96ce8985d

View file

@ -88,12 +88,16 @@ export class ElementCallOpenTelemetry {
} }
function recheckOTelEnabledStatus(optInAnalayticsEnabled: boolean): void { function recheckOTelEnabledStatus(optInAnalayticsEnabled: boolean): void {
if (optInAnalayticsEnabled && !sharedInstance) { const shouldEnable =
optInAnalayticsEnabled &&
Boolean(Config.get().opentelemetry?.collector_url);
if (shouldEnable && !sharedInstance) {
logger.info("Starting OpenTelemetry debug reporting"); logger.info("Starting OpenTelemetry debug reporting");
sharedInstance = new ElementCallOpenTelemetry( sharedInstance = new ElementCallOpenTelemetry(
Config.get().opentelemetry?.collector_url Config.get().opentelemetry?.collector_url
); );
} else if (!optInAnalayticsEnabled && sharedInstance) { } else if (!shouldEnable && sharedInstance) {
logger.info("Stopping OpenTelemetry debug reporting"); logger.info("Stopping OpenTelemetry debug reporting");
sharedInstance = undefined; sharedInstance = undefined;
} }