Skip to content

Commit

Permalink
Rename code_provenance.json reported by profiler to `code-provenanc…
Browse files Browse the repository at this point in the history
…e.json`

The profiling team decided to rename this file for consistency.

The code provenance feature (#1813) is not yet exposed to customers,
and the only release made with the old file name is 1.0.0.beta1 so
this does not cause any regression.
  • Loading branch information
ivoanjo committed Feb 25, 2022
1 parent 7bd0415 commit b7d503b
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions lib/datadog/profiling/ext.rb
Original file line number Diff line number Diff line change
Expand Up @@ -47,8 +47,8 @@ module HTTP
FORM_FIELD_PPROF_DATA = 'data[rubyprofile.pprof]'.freeze
PPROF_DEFAULT_FILENAME = 'rubyprofile.pprof.gz'.freeze

FORM_FIELD_CODE_PROVENANCE_DATA = 'data[code_provenance.json]'.freeze
CODE_PROVENANCE_FILENAME = 'code_provenance.json.gz'.freeze
FORM_FIELD_CODE_PROVENANCE_DATA = 'data[code-provenance.json]'.freeze
CODE_PROVENANCE_FILENAME = 'code-provenance.json.gz'.freeze
end
end
end
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -140,7 +140,7 @@

code_provenance_data = JSON.parse(
Datadog::Core::Utils::Compression.gunzip(
body.fetch('data[code_provenance.json]')
body.fetch('data[code-provenance.json]')
)
)

Expand Down
2 changes: 1 addition & 1 deletion spec/datadog/profiling/transport/http/api/endpoint_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@
call

expect(env.form)
.to include('data[code_provenance.json]' => kind_of(Datadog::Core::Vendor::Multipart::Post::UploadIO))
.to include('data[code-provenance.json]' => kind_of(Datadog::Core::Vendor::Multipart::Post::UploadIO))
end
end
end
Expand Down

0 comments on commit b7d503b

Please sign in to comment.