Skip to content

Commit 019a129

Browse files
committed
Consistent debugging and cleanup for named pipe dogstatsd code
1 parent a25a40d commit 019a129

File tree

9 files changed

+98
-211
lines changed

9 files changed

+98
-211
lines changed

.github/workflows/cargo.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -84,8 +84,6 @@ jobs:
8484
name: Test
8585
needs: setup
8686
runs-on: ${{ inputs.runner }}
87-
env:
88-
RUST_LOG: debug
8987
steps:
9088
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
9189
- uses: taiki-e/install-action@9ba3ac3fd006a70c6e186a683577abc1ccf0ff3a # v2.54.0

.gitignore

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,2 +1 @@
11
/target
2-
.idea

Cargo.lock

Lines changed: 2 additions & 154 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

LICENSE-3rdparty.csv

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -198,6 +198,7 @@ security-framework-sys,https://github.com/kornelski/rust-security-framework,MIT
198198
semver,https://github.com/dtolnay/semver,MIT OR Apache-2.0,David Tolnay <[email protected]>
199199
serde,https://github.com/serde-rs/serde,MIT OR Apache-2.0,"Erick Tryzelaar <[email protected]>, David Tolnay <[email protected]>"
200200
serde_bytes,https://github.com/serde-rs/bytes,MIT OR Apache-2.0,David Tolnay <[email protected]>
201+
serde_core,https://github.com/serde-rs/serde,MIT OR Apache-2.0,"Erick Tryzelaar <[email protected]>, David Tolnay <[email protected]>"
201202
serde_derive,https://github.com/serde-rs/serde,MIT OR Apache-2.0,"Erick Tryzelaar <[email protected]>, David Tolnay <[email protected]>"
202203
serde_json,https://github.com/serde-rs/json,MIT OR Apache-2.0,"Erick Tryzelaar <[email protected]>, David Tolnay <[email protected]>"
203204
serde_regex,https://github.com/tailhook/serde-regex,MIT OR Apache-2.0,[email protected]

crates/datadog-trace-agent/src/config.rs

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,12 +23,6 @@ pub struct Tags {
2323
function_tags_string: OnceLock<String>,
2424
}
2525

26-
impl Default for Tags {
27-
fn default() -> Self {
28-
Self::new()
29-
}
30-
}
31-
3226
impl Tags {
3327
pub fn from_env_string(env_tags: &str) -> Self {
3428
let mut tags = HashMap::new();

crates/datadog-trace-agent/src/trace_processor.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ impl TraceProcessor for ServerlessTraceProcessor {
101101
// double check content length is < max request content length in case transfer encoding is used
102102
if body_size > config.max_request_content_length {
103103
return log_and_create_http_response(
104-
"Error processing traces: Payload too large",
104+
&format!("Error processing traces: Payload too large"),
105105
StatusCode::PAYLOAD_TOO_LARGE,
106106
);
107107
}

crates/dogstatsd/Cargo.toml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,6 @@ zstd = { version = "0.13.3", default-features = false }
2727
datadog-fips = { path = "../datadog-fips", default-features = false }
2828

2929
[dev-dependencies]
30-
env_logger = "0.11"
3130
mockito = { version = "1.5.0", default-features = false }
3231
proptest = "1.4.0"
3332
tracing-test = { version = "0.2.5", default-features = false }

0 commit comments

Comments
 (0)