[log] Add debug logging to tracing/http.go WrapHTTPHandler

[github-actions]

Adds 4 meaningful debug log calls to internal/tracing/http.go using the existing logTracing logger declared in provider.go (same tracing package, no new logger declaration needed).

Changes

File: internal/tracing/http.go

Three categories of debug log calls added to WrapHTTPHandler:

1. Handler registration — logs when a handler is wired up with its OTel span name and the number of extra attributes:

   Registering HTTP handler with OTel span: span=mcp.tool_call, extraAttrs=2
   ```
   
   

2. Per-request entry — logs the span name, HTTP method, path, and whether an incoming W3C traceparent header was found (i.e., whether this is a continuation of a distributed trace or a fresh root trace):

   Handling request: span=mcp.tool_call, method=POST, path=/mcp, remoteParent=true
   

3. Post-span-start — logs the generated or propagated trace ID for cross-system correlation:

   Span started: span=mcp.tool_call, traceID=4bf92f3577b34da6a3ce929d0e0e4736
   

No Side Effects

• logTracing is already declared in provider.go in the same package — no new logger variable added
• oteltrace.SpanContextFromContext(ctx).IsRemote() is a pure read with no side effects
• span.SpanContext().TraceID() is a pure read; formats to hex via %s
• All logging is gated by DEBUG=tracing:* or DEBUG=* — zero overhead when disabled

Quality Checklist

• Exactly 1 file modified
• No test files modified
• Reuses existing logTracing logger (no duplicate declaration)
• Logger namespace follows pkg:filename convention (tracing:provider)
• Log arguments have no side effects
• Logging messages are meaningful and diagnostic
• No duplicate logging with existing log calls

Note

🔒 Integrity filter blocked 1 item

The following item were blocked because they don't meet the GitHub integrity level.

• search_code:open-telemetry/opentelemetry-go search_code: has lower integrity than agent requires. The agent cannot read data with integrity below "unapproved".

To allow these resources, lower min-integrity in your GitHub frontmatter:

tools:
  github:
    min-integrity: approved  # merged | approved | unapproved | none

Generated by Go Logger Enhancement · ◷

[Copilot]

Pull request overview

Adds additional debug-level observability to the tracing HTTP wrapper (tracing.WrapHTTPHandler) to help diagnose handler wiring and trace propagation behavior.

Changes:

• Log handler registration with span name and extra attribute count.
• Log per-request entry with method/path and whether an extracted remote parent is present.
• Log trace ID after starting the server span.

Show a summary per file

File	Description
internal/tracing/http.go	Adds debug logs around handler wrapping, request entry, and span start/trace ID for easier trace-correlation debugging.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

• Files reviewed: 1/1 changed files
• Comments generated: 1

[Copilot]

These new logs in http.go use logTracing which is namespaced as tracing:provider, so enabling/disabling debug output becomes misleading and you can’t selectively turn on tracing:http without also turning on provider logs. Elsewhere in the repo loggers are per-file/module (e.g. proxy:handler in internal/proxy/handler.go:24, proxy:graphql in internal/proxy/graphql.go:11); consider defining a tracing:http logger in this file and using it for request/handler logs.