Skip to content

Conversation

@trentm
Copy link
Contributor

@trentm trentm commented Dec 3, 2025

This adds support for using OTEL_SEMCONV_STABILITY_OPT_IN for controlled migration to stable net.* semconv. The net.* attributes are controlled by the http[/dup] token in OTEL_SEMCONV_STABILITY_OPT_IN (as discussed here).

Refs: open-telemetry/opentelemetry-js#5663

This adds support for using `OTEL_SEMCONV_STABILITY_OPT_IN` for controlled migration to stable `net.*` semconv. The `net.*` attributes are controlled by the `http[/dup]` token in `OTEL_SEMCONV_STABILITY_OPT_IN` (as [discussed here](open-telemetry/opentelemetry-js#5663 (comment))).

Refs: open-telemetry/opentelemetry-js#5663
@trentm
Copy link
Contributor Author

trentm commented Dec 3, 2025

Note: The tls.* attributes added to "tls-connect" spans are all waay out of date. It was added while tls.* semconv was being discussed. The currently TLS semconv has somewhat different values. However, as similar to this PR to instr-amqplib the tls.* semconv are not stable so we cannot use OTEL_SEMCONV_STABILITY_OPT_IN=http to handle the migration of those attributes.

These should still be updated, IMO. I have some notes on that. It is a TODO on me to open an issue for that.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants