Skip to content

Check how content-length is reported when zero content-length#3922

Merged
estringana merged 1 commit into
masterfrom
estringana/zero-content-length
Jun 9, 2026
Merged

Check how content-length is reported when zero content-length#3922
estringana merged 1 commit into
masterfrom
estringana/zero-content-length

Conversation

@estringana

Copy link
Copy Markdown
Contributor

Description

Adds AppSec integration coverage for http.response.headers.content-length on the root span, for both zero and non-zero values, and confirms the tag is always reported as a string (not a metric/number).

Reviewer checklist

  • Test coverage seems ok.
  • Appropriate labels assigned.

@datadog-official

datadog-official Bot commented May 27, 2026

Copy link
Copy Markdown

Pipelines  Tests

Fix all issues with BitsAI

⚠️ Warnings

🚦 5 Pipeline jobs failed

DataDog/apm-reliability/dd-trace-php | ZAI Shared Tests: [7.4]   View in Datadog   GitLab

🔧 Fix in code (Fix with Cursor). linking with `cc` failed: exit status: 1 due to realloc(): invalid pointer and LLVM ERROR: SmallVector unable to grow.

DataDog/apm-reliability/dd-trace-php | test_extension_ci: [8.0]   View in Datadog   GitLab

🔧 Fix in code (Fix with Cursor). 4 failed tests: Test dynamic config multiconfig priority merging, Enabling dynamic instrumentation via dynamic configuration, Installing a live debugger metric probe, Test exception replay.

DataDog/apm-reliability/dd-trace-php | test_extension_ci: [7.4]   View in Datadog   GitLab

🔄 Retry job. This looks flaky and may succeed on retry. 3 failed tests. Error: Installation timeouts and evaluation errors for live debugger probes.

View all 5 failed jobs.

ℹ️ Info

No other issues found (see more)

🧪 All tests passed
❄️ No new flaky tests detected

🎯 Code Coverage (details)
Patch Coverage: 100.00%
Overall Coverage: 60.70%

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: fd8a5e6 | Docs | Datadog PR Page | Give us feedback!

@estringana
estringana marked this pull request as ready for review May 28, 2026 08:00
@estringana
estringana requested a review from a team as a code owner May 28, 2026 08:00
@estringana
estringana merged commit 6a0bbf7 into master Jun 9, 2026
2124 of 2130 checks passed
@estringana
estringana deleted the estringana/zero-content-length branch June 9, 2026 15:14
@github-actions github-actions Bot added this to the 1.22.0 milestone Jun 9, 2026
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