buildkite / test-collector-ruby

Buildkite Test Analytics collector for Ruby test frameworks
http://buildkite.com/test-analytics
MIT License
15 stars 26 forks source link

Gzip body of upload request #183

Closed niceking closed 1 year ago

niceking commented 1 year ago

Gzip the payload for Upload API requests, and set Content-Encoding header so the server knows how to decode the payload. Relevant changes to bk/bk to support this is in PR #12006

niceking commented 1 year ago

@yob lol your review was so speedy that I hadn't even updated the other PR yet 😂

Yup I've done an integration test of PR #12006 against this change, and also sending the uncompressed payload as previously done.

Plan is to:

  1. Merge PR #12006 and check nothing breaks
  2. Merge PR #183
  3. Update Gemfile in buildkite with an updated git ref, and the branch that builds that will be sending the compressed data to a server that knows how to decompress that data