Update libdatadog to v30.0, make CA root optional for profiling#3758
Update libdatadog to v30.0, make CA root optional for profiling#3758morrisonlevi wants to merge 2 commits intomasterfrom
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #3758 +/- ##
==========================================
- Coverage 68.85% 68.84% -0.02%
==========================================
Files 166 166
Lines 19015 19015
Branches 1792 1792
==========================================
- Hits 13093 13091 -2
- Misses 5111 5114 +3
+ Partials 811 810 -1
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
Benchmarks [ profiler ]Benchmark execution time: 2026-03-30 21:16:17 Comparing candidate commit e45f220 in PR branch Found 2 performance improvements and 0 performance regressions! Performance is the same for 26 metrics, 8 unstable metrics. scenario:php-profiler-timeline-memory-with-profiler-and-timeline
|
|
✅ Tests 🎉 All green!❄️ No new flaky tests detected 🎯 Code Coverage (details) 🔗 Commit SHA: e45f220 | Docs | Datadog PR Page | Was this helpful? React with 👍/👎 or give us feedback! |
Description
This supersedes #3734. In a recent libdatadog change, we unintentionally required that a CA root store be found even for non-HTTPS connections to the agent e.g.
http://andunix://endpoints.This PR targets the v30 release of libdatadog which includes the fix. It does this by updating the vendored
libdatadog/which was previously pointing to a commit on main to point to v30.0.0, and then updating theprofiling/crate to usepath = "../libdatdog/....Reviewer checklist