Merge pull request #124 from zasdaym/fix/configure-opentelemetry

vmihailenco · web-flow · commit 9ec993e31894 · 2022-07-23T09:21:30.000+03:00
fix: opentelemetry exit condition
diff --git a/uptrace/uptrace.go b/uptrace/uptrace.go
@@ -27,7 +27,7 @@ func ConfigureOpentelemetry(opts ...Option) {
 	ctx := context.TODO()
 	cfg := newConfig(opts)
 
-	if !cfg.tracingEnabled && cfg.metricsEnabled {
+	if !cfg.tracingEnabled && !cfg.metricsEnabled {
 		return
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ func ConfigureOpentelemetry(opts ...Option) {`
`27`	`27`	`ctx := context.TODO()`
`28`	`28`	`cfg := newConfig(opts)`
`29`	`29`
`30`		`- if !cfg.tracingEnabled && cfg.metricsEnabled {`
	`30`	`+ if !cfg.tracingEnabled && !cfg.metricsEnabled {`
`31`	`31`	`return`
`32`	`32`	`}`
`33`	`33`