-
Notifications
You must be signed in to change notification settings - Fork 75
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
The uploader receives bad requests when setting CI env in GitHub actions #217
Comments
Getting the same with
|
@TriplEight do you mind opening a separate issue for this if you are not using GitHub Actions? I think it's a different cause, and I'd like to be able to keep track of the issues to make sure they are taken care of. |
@dustin71728, I made a change to the uploader, are you still experiencing the issue? |
Thanks for the effort, I can confirm that GITHUB_SERVER_URL is produced by
|
awesome @dustin71728, closing this out |
Describe the bug
When I run the codecov uploader in the docker container, everything is fine until I set up environment variables using the first technique mentioned in the following article:
https://docs.codecov.com/docs/testing-with-docker
Then the uploader complains about bad requests:
I wonder if the root cause is that the build_url query parameter is not a well-formatted URL. It is a literal undefined string in the host part of the URL.
So I try to set up CI_BUILD_URL with the correct value, but it still doesn't work.
The failed log( In
Run yarn test
step):https://github.com/dustin71728/yarn-2-completion/runs/3091016225?check_suite_focus=true
The log printed out all the environment variables returned from the command:
"ci_env=
bash <(curl -s https://codecov.io/env)
"https://github.com/dustin71728/yarn-2-completion/runs/3090999690?check_suite_focus=true
The successful log without CI environment variables involved:
https://github.com/dustin71728/yarn-2-completion/runs/3081701233?check_suite_focus=true
References
Dockerfile: https://github.com/dustin71728/yarn-2-completion/blob/test/migrate-to-shellspec/docker/bash-5-0.Dockerfile
Github Action YAML file: https://github.com/dustin71728/yarn-2-completion/blob/test/migrate-to-shellspec/.github/workflows/default.yaml
Expected behavior
The uploader should upload the coverage report when the CI is true in the docker container running on Github action.
The text was updated successfully, but these errors were encountered: