config: NewSDK can return valid TracerProvider

config/trace.go

@@ -30,7 +30,7 @@
 	for _, processor := range cfg.opentelemetryConfig.TracerProvider.Processors {
 		sp, err := spanProcessor(cfg.ctx, processor)
 		if err == nil {
 			opts = append(opts, sdktrace.WithSpanProcessor(sp))
 		} else {
 			errs = append(errs, err)
 		}
@@ -94,6 +94,11 @@
 		if err != nil {
 			return nil, err
 		}
+		// ParseRequestURI leaves the Host field empty when no
+		// scheme is specified (i.e. localhost:4317). This check is
+		// here to support the case where a user may not specify a
+		// scheme. The code does its best effort here by using
+		// otlpConfig.Endpoint as-is in that case
 		if u.Host != "" {
 			opts = append(opts, otlptracegrpc.WithEndpoint(u.Host))
 		} else {
@@ -109,7 +114,7 @@
 		switch *otlpConfig.Compression {
 		case compressionGzip:
 			opts = append(opts, otlptracegrpc.WithCompressor(*otlpConfig.Compression))
 		case compressionNone:
 			// none requires no options
 		default:
 			return nil, fmt.Errorf("unsupported compression %q", *otlpConfig.Compression)