Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Regression v5.0.7: Unable to upload coverage report for PRs coming from forked repos #1705

Closed
audgirka opened this issue Nov 26, 2024 · 0 comments · Fixed by #1724
Closed

Regression v5.0.7: Unable to upload coverage report for PRs coming from forked repos #1705

audgirka opened this issue Nov 26, 2024 · 0 comments · Fixed by #1724
Assignees
@thomasrockhu-codecov
Labels
Area: Report Upload Issues with pre-ingest report uploading high High Priority Issues (to be fixed within 2 sprints)

Comments

@audgirka
Copy link

When trying to upload coverage report after the CI jobs are complete we get a failure for URL missing or bad / illegal URL.
Refer job

This PR 4422 is coming from a forked repo.

Noticed that this does not happen for PRs from the same repo.
@audgirka audgirka mentioned this issue Nov 26, 2024
Merged
@rohan-at-sentry rohan-at-sentry added the Area: Report Upload Issues with pre-ingest report uploading label Nov 26, 2024
This was referenced Nov 26, 2024
Merged
Closed
@wdconinc wdconinc mentioned this issue Nov 29, 2024
Closed
@thomasrockhu-codecov thomasrockhu-codecov added the high High Priority Issues (to be fixed within 2 sprints) label Dec 5, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@thomasrockhu-codecov thomasrockhu-codecov

Labels
Area: Report Upload Issues with pre-ingest report uploading high High Priority Issues (to be fixed within 2 sprints)
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: prevent oidc on tokenless due to permissioning codecov/codecov-action
3 participants
@audgirka @thomasrockhu-codecov @rohan-at-sentry