Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

beater/api/profile: fix pprof Content-Type #3144

Merged
merged 1 commit into from
Jan 13, 2020

Conversation

axw
Copy link
Member

@axw axw commented Jan 13, 2020

If messageType is specified, check that it has the
expected value for pprof. Otherwise we assume
pprof, and optimistically attempt to decode.

Closes #2954

If messageType is specified, check that it is the
expected value. Otherwise we assume pprof, and
optimistically attempt to decode.
@axw axw force-pushed the profile-content-type branch from 87aa04c to c792d6f Compare January 13, 2020 03:26
@codecov-io
Copy link

Codecov Report

Merging #3144 into master will increase coverage by 0.02%.
The diff coverage is 73.33%.

@@            Coverage Diff            @@
##           master   #3144      +/-   ##
=========================================
+ Coverage   79.68%   79.7%   +0.02%     
=========================================
  Files         105     105              
  Lines        5547    5554       +7     
=========================================
+ Hits         4420    4427       +7     
  Misses       1127    1127
Impacted Files Coverage Δ
beater/api/profile/handler.go 82.78% <73.33%> (+1.04%) ⬆️

@axw axw requested a review from a team January 13, 2020 05:46
@axw axw merged commit 1e2b099 into elastic:master Jan 13, 2020
@axw axw deleted the profile-content-type branch January 13, 2020 12:10
axw added a commit to axw/apm-server that referenced this pull request Jan 13, 2020
If messageType is specified, check that it is the
expected value. Otherwise we assume pprof, and
optimistically attempt to decode.
axw added a commit that referenced this pull request Jan 14, 2020
If messageType is specified, check that it is the
expected value. Otherwise we assume pprof, and
optimistically attempt to decode.
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.

beater/api/profile: fix pprof content-type
3 participants