fix: don't send all files in header (#81)

coverallsapp · Jun 1, 2023 · a37c1b4 · a37c1b4
1 parent 6e7d716
commit a37c1b4
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 7 deletions.
diff --git a/spec/coverage_reporter/api/jobs_spec.cr b/spec/coverage_reporter/api/jobs_spec.cr
@@ -42,7 +42,6 @@ Spectator.describe CoverageReporter::Api::Jobs do
         "X-Coveralls-Reporter"         => "coverage-reporter",
         "X-Coveralls-Reporter-Version" => CoverageReporter::VERSION,
         "X-Coveralls-Coverage-Formats" => "cobertura",
-        "X-Coveralls-Files"            => "cobertura.xml",
         "X-Coveralls-Source"           => "cli",
       },
       body: {

diff --git a/src/coverage_reporter/api/jobs.cr b/src/coverage_reporter/api/jobs.cr
@@ -24,21 +24,23 @@ module CoverageReporter
       data = build_request
       api_url = "#{@config.endpoint}/api/#{API_VERSION}/jobs"
 
+      headers = DEFAULT_HEADERS.merge({
+        "Content-Type"                 => "application/json",
+        "X-Coveralls-Coverage-Formats" => @source_files.map(&.format.to_s).sort!.uniq!.join(","),
+        "X-Coveralls-CI"               => @config[:service_name]?,
+      }.compact)
+
       Log.info "  ·job_flag: #{@config.flag_name}" if @config.flag_name
       Log.info "🚀 Posting coverage data to #{api_url}"
 
+      Log.debug "---\n⛑ Debug Headers:\n#{headers.to_pretty_json}"
       Log.debug "---\n⛑ Debug Output:\n#{data.to_pretty_json}"
 
       return if dry_run
 
       res = Crest.post(
         api_url,
-        headers: DEFAULT_HEADERS.merge({
-          "Content-Type"                 => "application/json",
-          "X-Coveralls-Coverage-Formats" => @source_files.map(&.format.to_s).sort!.uniq!.join(","),
-          "X-Coveralls-Files"            => @source_files.filenames.join(","),
-          "X-Coveralls-CI"               => @config[:service_name]?,
-        }.compact),
+        headers: headers,
         form: {:json => data.to_json.to_s}.to_json,
         tls: ENV["COVERALLS_ENDPOINT"]? ? OpenSSL::SSL::Context::Client.insecure : nil
       )