ARCHIVE/rippled
1
0
Fork 0
mirror of https://github.com/XRPLF/rippled.git synced 2026-08-21 22:30:57 +00:00
Code Issues Packages Projects Releases Wiki Activity

docs(telemetry): drop plan-document pointers from consensus tracing comments

Browse Source 
These comments pointed at a planning folder and at its rollout phase
numbering, neither of which is part of the shipped tree, so the
references would dangle for any reader of the repository. Each comment
now states the fact it was pointing at.
This commit is contained in:
Pratik Mankawde
2026-08-14 21:11:07 +01:00
parent 2e21758a49
commit 6ac68abfb5
4 changed files with 12 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
include/xrpl/telemetry/TraceContextPropagator.h
View File

@@ -101,13 +101,10 @@ injectToProtobuf(opentelemetry::context::Context const& ctx, protocol::TraceCont
// Serialize flags
proto.set_trace_flags(spanCtx.trace_flags().flags());
// TODO(observability/secure-OTel): the protobuf TraceContext message
// also carries `trace_state` (field 4), which is currently neither
// populated here nor read by extractFromProtobuf above. The field is
// reserved for the secure tracing pipeline outlined in
// OpenTelemetryPlan/secure-OTel.md, where an authenticated token in
// tracestate will let receivers reject spoofed/poisoned trace context.
// Wire trace_state through inject/extract once the consumer lands.
// TODO: the protobuf TraceContext message also carries `trace_state`
// (field 4), which is currently neither populated here nor read by
// extractFromProtobuf above. The field is reserved for future use;
// wire it through inject/extract once a consumer lands.
}
} // namespace xrpl::telemetry