Skip to content

Conversation

duncanista
Copy link

What does this PR do?

Exports the datadog module and some constants

Motivation

Need to replace the code in the Datadog Lambda Extension (precursor of this module) to use this crate

Additional Notes

n/a

Comment on lines +24 to +30
pub const DATADOG_HIGHER_ORDER_TRACE_ID_BITS_KEY: &str = "_dd.p.tid";
pub const DATADOG_TRACE_ID_KEY: &str = "x-datadog-trace-id";
pub const DATADOG_ORIGIN_KEY: &str = "x-datadog-origin";
pub const DATADOG_PARENT_ID_KEY: &str = "x-datadog-parent-id";
pub const DATADOG_SAMPLING_PRIORITY_KEY: &str = "x-datadog-sampling-priority";
pub const DATADOG_TAGS_KEY: &str = "x-datadog-tags";
pub const DATADOG_PROPAGATION_ERROR_KEY: &str = "_dd.propagation_error";
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wondering if it would be better to export some of this as constants so that the import is not dd_trace_propgation::datadog::{...} but just dd_trace_propagation::constants::{...}

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's keep them in the datadog namespace.

Comment on lines +24 to +30
pub const DATADOG_HIGHER_ORDER_TRACE_ID_BITS_KEY: &str = "_dd.p.tid";
pub const DATADOG_TRACE_ID_KEY: &str = "x-datadog-trace-id";
pub const DATADOG_ORIGIN_KEY: &str = "x-datadog-origin";
pub const DATADOG_PARENT_ID_KEY: &str = "x-datadog-parent-id";
pub const DATADOG_SAMPLING_PRIORITY_KEY: &str = "x-datadog-sampling-priority";
pub const DATADOG_TAGS_KEY: &str = "x-datadog-tags";
pub const DATADOG_PROPAGATION_ERROR_KEY: &str = "_dd.propagation_error";
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's keep them in the datadog namespace.

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.

2 participants