-
Notifications
You must be signed in to change notification settings - Fork 213
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 valid tokenless upload #1548
Comments
We're seeing this on forks too:
|
Yeah, sometimes it works, some other times it doesn't. That's why I wrongly thought doctrine/dbal#6501 fixed the issue. |
cc @derrabus |
@Dreamsorcerer the workaround is to have the clear text token in the config. |
I'm getting this when using OIDC, is that expected? |
@greg0ire @Dreamsorcerer @webknjaz @ofek sorry for the delay here. I'm hoping that this has been remedied in v5. Are you all still experiencing this issue? |
I would say it is working most of the time now, but I've seen it failed once. That's an improvement though, because before, it was failing most of the time. |
Seems to be working now 🤞 |
Not sure if it's an unrelated issue, but I have a PR from a fork which has used tokenless uploading. aio-libs/aiohttp-session#1047 |
It seems really different, I'd file a separate issue. |
Yeah, I got confused by the link which says that it might be caused by failing to upload reports. Actually, I think the issue is that the previous commit it's trying to compare against on master is a Dependabot commit which didn't upload any reports. It should probably fallback to comparing the latest commit it does have uploads for... |
@Dreamsorcerer @greg0ire is correct, different issue. As for pulling uploads, I think you're looking for something like carryforward flags |
As asked in #1487 (comment), I'm opening a new ticket.
The failure is still happening for tokenless uploads for
doctrine/dbal
. What's weird is that I've seen it work on one of my own PRs, made from my fork.Error message
error - 2024-08-17 09:46:06,912 -- Report creating failed: {"detail":"Not valid tokenless upload"}
Job definition
Links
The text was updated successfully, but these errors were encountered: