-
Notifications
You must be signed in to change notification settings - Fork 6
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
"Not a valid tokenless upload" error not relevant #366
Comments
Personally, I'm half guessing codecov made a call to https://api.github.com/ -- I'd really like to know what api call it made (and at least what identity it used -- I obviously don't want the credentials...) As an end user, from my perspective, the repository is public
So, |
Hi @jsoref , We just merged a fix for the OIDC, can you please try again? |
@drazisil-codecov: to which repository? (Do I need to update my fork of the codecov-action repository?) |
Hi @jsoref I believe it was to API so no updates should be needed on your side. I'm confirming with the engineer. |
@drazisil-codecov here's a new (failed) run from the same commit: https://github.com/check-spelling-sandbox/eslint-plugin-jest/actions/runs/8986717338/job/24776595047 |
I have passed the details back to the engineer. |
@jsoref it's me again, I think I have found the root cause here, thanks for being patient. We'll work on a fix. |
@thomasrockhu-codecov: I appreciate fixing the underlying error, but it'd really be nice if the error message itself were improved. Anyway, @ me when you want me to try again. I'm usually fairly responsive 🙂 . |
@jsoref yeah, I guess the problem is more that you should not have seen that error message to begin with. Definitely a bug on our side |
Fwiw, here's a round where the codecov-action is now configured to self-test using oidc: https://github.com/check-spelling-sandbox/codecov-action/actions/runs/9116103249/job/25063948144 on the bright side, it fails pretty fast. |
@jsoref I believe we made a fix for this, are you still seeing |
I'm not seeing that error. https://github.com/check-spelling-sandbox/eslint-plugin-jest/actions/runs/9715692458/job/26817680471 has |
@jsoref ok, I think we pushed a fix for that, do you mind re-running CI? |
I don't see a change? https://github.com/check-spelling-sandbox/eslint-plugin-jest/actions/runs/9715692458 |
Repository not found
isn't helpful
Describe the bug
Some "http server" is yielding
HTTP Error 400
with a message ofRepository not found
. This isn't actionable. Nor is there any content on https://docs.codecov.com/docs/ for this messageEnvironment (please complete the following information):
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Clear error messages explaining what was done, what was seen, and why that results in a specific error.
Screenshots
log
Additional context
The text was updated successfully, but these errors were encountered: