Skip to content

Commit

Permalink
start telemetry client after appsec started
Browse files Browse the repository at this point in the history
Signed-off-by: Eliott Bouhana <[email protected]>
  • Loading branch information
eliottness committed Feb 19, 2025
1 parent 1a2e7dd commit a293e49
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions ddtrace/tracer/tracer.go
Original file line number Diff line number Diff line change
Expand Up @@ -186,10 +186,6 @@ func Start(opts ...StartOption) {
log.Warn("Remote config startup error: %s", err)
}

// start instrumentation telemetry unless it is disabled through the
// DD_INSTRUMENTATION_TELEMETRY_ENABLED env var
startTelemetry(t.config)

// appsec.Start() may use the telemetry client to report activation, so it is
// important this happens _AFTER_ startTelemetry() has been called, so the
// client is appropriately configured.
Expand All @@ -198,6 +194,10 @@ func Start(opts ...StartOption) {
appsecopts = append(appsecopts, appsecConfig.WithRCConfig(cfg), appsecConfig.WithMetaStructAvailable(t.config.agent.metaStructAvailable))
appsec.Start(appsecopts...)

// start instrumentation telemetry unless it is disabled through the
// DD_INSTRUMENTATION_TELEMETRY_ENABLED env var
startTelemetry(t.config)

if t.config.logStartup {
logStartup(t)
}
Expand Down

0 comments on commit a293e49

Please sign in to comment.