fix: (*configx.Provider).TracingConfig should parse all configs

ory · Sep 19, 2022 · b57a9bf · b57a9bf
1 parent 56ef9fb
commit b57a9bf
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 6 deletions.
diff --git a/configx/provider.go b/configx/provider.go
@@ -489,6 +489,19 @@ func (p *Provider) TracingConfig(serviceName string) *otelx.Config {
 				},
 				LocalAgentAddress: p.String("tracing.providers.jaeger.local_agent_address"),
 			},
+			Zipkin: otelx.ZipkinConfig{
+				ServerURL: p.String("tracing.providers.zipkin.server_url"),
+				Sampling: otelx.ZipkinSampling{
+					SamplingRatio: p.Float64("tracing.providers.zipkin.sampling.sampling_ratio"),
+				},
+			},
+			OTLP: otelx.OTLPConfig{
+				ServerURL: p.String("tracing.providers.otlp.server_url"),
+				Insecure:  p.Bool("tracing.providers.otlp.insecure"),
+				Sampling: otelx.OTLPSampling{
+					SamplingRatio: p.Float64("tracing.providers.otlp.sampling.sampling_ratio"),
+				},
+			},
 		},
 	}
 }

diff --git a/otelx/config.schema.json b/otelx/config.schema.json
@@ -88,9 +88,7 @@
               "description": "The address of the Zipkin server where spans should be sent to.",
               "format": "uri",
               "examples": [
-                {
-                  "server_url": "http://localhost:9411/api/v2/spans"
-                }
+                "http://localhost:9411/api/v2/spans"
               ]
             },
             "sampling": {
@@ -136,9 +134,7 @@
                 }
               ],
               "examples": [
-                {
-                  "server_url": "localhost:4318"
-                }
+                "localhost:4318"
               ]
             },
             "insecure": {