Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clarify doc comment for DD_TRACE_128_BIT_TRACEID_LOGGING_ENABLED #6791

Merged
merged 3 commits into from
Mar 21, 2025
+5 −3
Merged

Clarify doc comment for DD_TRACE_128_BIT_TRACEID_LOGGING_ENABLED #6791

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
704cbcb
Clarify doc comment for DD_TRACE_128_BIT_TRACEID_LOGGING_ENABLED
zacharycmontoya Mar 20, 2025
cf77e5e
Another small refinement
zacharycmontoya Mar 20, 2025
8ec47fa
Update tracer/src/Datadog.Trace/Configuration/ConfigurationKeys.cs
zacharycmontoya Mar 21, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 5 additions & 3 deletions tracer/src/Datadog.Trace/Configuration/ConfigurationKeys.cs
View file
Original file line number Diff line number Diff line change
@@ -805,9 +805,11 @@ internal static class FeatureFlags
/// If disabled, 128-bit trace ids will be truncated to the lower 64 bits,
/// and injected as decimal strings. 64-bit trace ids are
/// always injected as decimal strings, regardless of this setting.
/// Default value is <c>true</c> (enabled).
/// If <see cref="ConfigurationKeys.FeatureFlags.TraceId128BitGenerationEnabled"/> is set to <c>false</c>,
/// this setting will also be set to <c>false</c>.
/// If unset, this configuration will take the value of the <see cref="ConfigurationKeys.FeatureFlags.TraceId128BitGenerationEnabled"/> configuration,
/// which is <c>true</c> by default.
/// Note: This configuration can be set independently of the <see cref="ConfigurationKeys.FeatureFlags.TraceId128BitGenerationEnabled"/> configuration,
/// so it's possible to inject 128-bit trace ids into logs even if the application is only generating 64-bit trace ids, since distributed traces from upstream
/// services may contain 128-bit trace ids.
/// </summary>
public const string TraceId128BitLoggingEnabled = "DD_TRACE_128_BIT_TRACEID_LOGGING_ENABLED";

Loading