fix: bug in the order of options

Author: hweawer

utils/httputil/httputil.go

@@ -135,14 +135,15 @@ func IsNetworkError(err error) bool {
 }
 
 type sendOptions struct {
-	body          io.Reader
-	timeout       time.Duration
-	acceptedCodes map[int]bool
-	headers       map[string]string
-	redirect      func(req *http.Request, via []*http.Request) error
-	retry         retryOptions
-	transport     http.RoundTripper
-	ctx           context.Context
+	body           io.Reader
+	timeout        time.Duration
+	acceptedCodes  map[int]bool
+	headers        map[string]string
+	redirect       func(req *http.Request, via []*http.Request) error
+	retry          retryOptions
+	transport      http.RoundTripper
+	ctx            context.Context
+	tracingContext bool
 
 	// This is not a valid http option. It provides a way to override
 	// parts of the url. For example, url.Scheme can be changed from
@@ -280,17 +281,13 @@ func SendContext(ctx context.Context) SendOption {
 	return func(o *sendOptions) { o.ctx = ctx }
 }
 
-// SendTracingContext sets the context and wraps the HTTP transport with otelhttp
-// for automatic trace context propagation via HTTP headers.
+// SendTracingContext sets the context and enables OpenTelemetry trace context
+// propagation via HTTP headers. The transport will be wrapped with otelhttp
+// after all options are applied.
 func SendTracingContext(ctx context.Context) SendOption {
 	return func(o *sendOptions) {
 		o.ctx = ctx
-		// Wrap the existing transport (or default) with otelhttp
-		baseTransport := o.transport
-		if baseTransport == nil {
-			baseTransport = http.DefaultTransport
-		}
-		o.transport = otelhttp.NewTransport(baseTransport)
+		o.tracingContext = true
 	}
 }
 
@@ -315,6 +312,16 @@ func Send(method, rawurl string, options ...SendOption) (*http.Response, error)
 		o(opts)
 	}
 
+	// Apply tracing context wrapping AFTER all other options are processed
+	// This ensures SendTLS and other transport options are respected
+	if opts.tracingContext {
+		baseTransport := opts.transport
+		if baseTransport == nil {
+			baseTransport = http.DefaultTransport
+		}
+		opts.transport = otelhttp.NewTransport(baseTransport)
+	}
+
 	req, err := newRequest(method, opts)
 	if err != nil {
 		return nil, err