Skip to content

Conversation

grainrigi
Copy link
Contributor

What does this PR do?

Enable to use continuous profiler with a https datadog agent.

Motivation

To fix #3190

Additional Notes

I've confirmed that profiling data is correctly sent to the agent with this fix.

@grainrigi grainrigi requested a review from a team as a code owner May 27, 2023 13:05
@codecov
Copy link

codecov bot commented Jun 13, 2023

Codecov Report

Merging #3194 (e136ea9) into master (58d2e20) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #3194      +/-   ##
==========================================
+ Coverage   84.27%   84.28%   +0.01%     
==========================================
  Files         214      214              
  Lines        8398     8400       +2     
  Branches       33       33              
==========================================
+ Hits         7077     7080       +3     
+ Misses       1321     1320       -1     
Impacted Files Coverage Δ
packages/dd-trace/src/profiling/exporters/agent.js 98.75% <100.00%> (+0.03%) ⬆️

... and 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@szegedi szegedi merged commit 257294b into DataDog:master Jul 19, 2023
szegedi added a commit that referenced this pull request Jul 20, 2023
szegedi added a commit that referenced this pull request Jul 20, 2023
szegedi added a commit that referenced this pull request Jul 20, 2023
tlhunter pushed a commit that referenced this pull request Jul 21, 2023
tlhunter pushed a commit that referenced this pull request Jul 21, 2023
tlhunter pushed a commit that referenced this pull request Jul 21, 2023
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.

Continuous Profiler doesn't work with a datadog agent serving on HTTPS
6 participants