coverallsapp / coverage-reporter

Self-contained, universal coverage uploader binary. Under development.
https://coveralls.io
MIT License
45 stars 14 forks source link

Crest::BadRequest using Coveralls > v0.3.5 #80

Closed pvheiningen closed 1 year ago

pvheiningen commented 1 year ago

What was wrong?

Hi there, we're using the universal coverage reporter to upload gcov coverage files to coveralls.io. We have some issues with newer versions. Any version above 0.3.5 gives me the <Crest::BadRequest:HTTP status code 400: Bad Request> error.

v0.3.5:

🚀 Posting coverage data to https://coveralls.io/api/v1/jobs
---
✅ API Response: {"message":"Job ##11.1","url":"https://coveralls.io/jobs/122307634"}
- 💛, Coveralls

v0.3.6:

🚀 Posting coverage data to https://coveralls.io/api/v1/jobs
#<Crest::BadRequest:HTTP status code 400: Bad Request>
Unhandled exception: HTTP status code 400: Bad Request (Crest::BadRequest)
  from ???
  from ???
  from ???
  from ???
  from ???
  from ???
  from ???
  from ???

I believe the problem is potentially caused by the request being too large, if I limit the number of gcov files that we're uploading it starts working again on v0.3.6.

mrexox commented 1 year ago

Hey! Thank you for the issue. I have a few questions to better understand the issue:

It looks like the problem is on a backend side, maybe it's too big payload. Could you share your gcov coverage file, please? You can send it on my email - valentin@coveralls.io, it would be the best way for me to find the issue.

pvheiningen commented 1 year ago

Thanks for the quick response!

Sent you an email with the actual gcov files, hopefully you can use those to reproduce.

Thanks in advance!

mrexox commented 1 year ago

Hey! Thank you very much for sharing the reports. I found an issue and I'm releasing a fix in v0.4.4 of coverage reporter. If you use Github action or download the latest binary, nothing should be done on your side, just re-run the workflow. Otherwise, specify v0.4.4 version for the new workflows, please.

pvheiningen commented 1 year ago

Thanks a lot for the quick turnaround! I appreciate it! :tada: