Skip to content

Conversation

@bojbrook
Copy link
Contributor

@bojbrook bojbrook commented Jan 15, 2026

What does this PR do?

Adds a test to the http integration on our handling to many concurrent http requests.

Motivation

From working with @BridgeAR on issue APMS-16671 where the customer was hitting the max requests limit of the tracer

Plugin Checklist

Additional Notes

@github-actions
Copy link

github-actions bot commented Jan 15, 2026

Overall package size

Self size: 4.39 MB
Deduped: 5.21 MB
No deduping: 5.21 MB

Dependency sizes | name | version | self size | total size | |------|---------|-----------|------------| | import-in-the-middle | 2.0.0 | 68.46 kB | 797.03 kB | | dc-polyfill | 0.1.10 | 26.73 kB | 26.73 kB |

🤖 This report was automatically generated by heaviest-objects-in-the-universe

@datadog-datadog-prod-us1
Copy link

datadog-datadog-prod-us1 bot commented Jan 15, 2026

⚠️ Tests

Fix all issues with Cursor

⚠️ Warnings

🧪 2 Tests failed

IAST - overhead-controller - integration "after each" hook for "should report vulnerability only in the first request" from vulnerability sampling algorithm (Datadog) (Fix with Cursor)
Cannot read properties of undefined (reading 'kill')

TypeError: Cannot read properties of undefined (reading 'kill')
    at Context.<anonymous> (packages/dd-trace/test/appsec/iast/overhead-controller.integration.spec.js:27:10)
    at process.processImmediate (node:internal/timers:504:21)
IAST - overhead-controller - integration vulnerability sampling algorithm "before each" hook for "should report vulnerability only in the first request" from vulnerability sampling algorithm (Datadog) (Fix with Cursor)
Timeout of 5000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/Users/runner/work/dd-trace-js/dd-trace-js/packages/dd-trace/test/appsec/iast/overhead-controller.integration.spec.js)

Error: Timeout of 5000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/Users/runner/work/dd-trace-js/dd-trace-js/packages/dd-trace/test/appsec/iast/overhead-controller.integration.spec.js)
    at listOnTimeout (node:internal/timers:605:17)
    at process.processTimers (node:internal/timers:541:7)

ℹ️ Info

❄️ No new flaky tests detected

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 373d21d | Docs | Datadog PR Page | Was this helpful? Give us feedback!

@codecov
Copy link

codecov bot commented Jan 15, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.13%. Comparing base (033993d) to head (373d21d).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #7256   +/-   ##
=======================================
  Coverage   85.13%   85.13%           
=======================================
  Files         532      532           
  Lines       22654    22654           
=======================================
  Hits        19287    19287           
  Misses       3367     3367           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@pr-commenter
Copy link

pr-commenter bot commented Jan 15, 2026

Benchmarks

Benchmark execution time: 2026-01-15 18:50:23

Comparing candidate commit 373d21d in PR branch bojbrook/http-slowagent-test with baseline commit 033993d in branch master.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 291 metrics, 29 unstable metrics.

@bojbrook bojbrook marked this pull request as ready for review January 15, 2026 19:17
@bojbrook bojbrook requested a review from a team as a code owner January 15, 2026 19:17
@bojbrook bojbrook requested a review from BridgeAR January 15, 2026 19:17
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