-
Notifications
You must be signed in to change notification settings - Fork 17
Getting 403 response from GitHub #183
Comments
Hi @cglewis, do you have a link to your repo pls? |
@cglewis Ok just found it's https://github.com/CyberReboot/poseidon. I will try to reproduce with your workflow. Keep you in touch. |
@cglewis Cannot reproduce with this workflow. There is maybe a limitation if your org triggers to much calls to the download release endpoint. I'll see if it's possible to use the GITHUB_TOKEN to lift this limit if that's the case. |
Huh. I've never run into limits before. Interesting. I'll try it out later and see if resolves itself. Thanks for looking into it! |
Me neither and I download assets through many other actions I made so I'm pretty worried about it 😅
Sure you're welcome |
@crazy-max I just retriggered the workflow, to see if it worked, and seems happy again. Could have been a GitHub/S3 issue. I'm going to close this for now, but will reopen if I run into the issue again. Thanks again for an awesome project and the quick response! |
I was going to say the same thing to you, it seems to happen with the HEAD response returning a 403 from S3 indeed. Let's keep an eye on it. I'll do some more research of my own.
🤝 |
@cglewis Next time you've got this issue, please add the secret |
@cglewis Ok this is an incident with GitHub. |
Ahh! Nice find! I actually looked there first, and it was all green at the time! |
To avoid this problem in the future, I will use the runner cache system to avoid re-downloading buildx at each build if the version did not changed. |
@cglewis Cache of Buildx binary is now implemented. |
I'm using this on several projects and they all started doing this just today:
The yaml for that step is pretty vanilla:
I just updated to 1.6.0 today, it's also broken on 1.4.0 which is what I was using before.
The text was updated successfully, but these errors were encountered: