Skip to content

[chore][processor/redactionprocessor] Updating to latest semconv version#49467

Draft
anarwal wants to merge 1 commit into
open-telemetry:mainfrom
anarwal:processor/redactionprocessor
Draft

[chore][processor/redactionprocessor] Updating to latest semconv version#49467
anarwal wants to merge 1 commit into
open-telemetry:mainfrom
anarwal:processor/redactionprocessor

Conversation

@anarwal

@anarwal anarwal commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

Description

  1. internal/url/spanname.go — Removed semconv125 import, renamed semconv138 to conventions. Replaced old HTTP constants (HTTPSchemeKey, HTTPTargetKey, HTTPMethodKey, HTTPURLKey) with string literals. The code already checks both old and new HTTP keys for dual-support.
  2. internal/db/spanname.go — Removed semconv128 import, renamed semconv138 to conventions. Replaced semconv128.DBSystemKey with "db.system". The code already checks db.system.name (new) first, falling back to db.system (old).

Link to tracking issue

Fixes processor/redactionprocessor from #22095

Testing

Tested by go mod tidy && go build ./... && go test ./... from processor/redactionprocessor

Authorship

  • I, a human, wrote this pull request description myself.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant