-
Notifications
You must be signed in to change notification settings - Fork 736
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
[BUG] Sudden issue with Dependabot secrets #1801
Comments
@nikolay what version of the provider are you using? Can we narrow it down to a specific pull request, or do you think it's an API-side change? |
@kfcampbell Always the latest and greatest. I downgraded to v5.29, but it seems to be a GitHub API issue, I guess. |
@kfcampbell Justed tested with v5.32 - it's even worse! On top of the old issue, now the "inconsistent results" error adds up. |
Hmm...I'm not seeing any particularly suspicious PRs to this repo recently, are you? |
As the ticket states, it just suddenly stopped working, i.e. something has changed on the GitHub's side, I guess. |
In that case, perhaps the best thing to do would be report it to the API team here. |
@kfcampbell The API is fine and works great - it's the provider you maintain or the Go library that's not using it correctly. The error message is pretty clear - you want a string to be a string, which it is, though you have a problem with strings being strings. |
I wonder if it's related to google/go-github#2817. There hasn't been a new release cut since that PR went in, but we can update ASAP as soon as there is one. |
@kfcampbell Most probably if Dependabot secrets are covered by that as the issue I'm getting is specifically with Dependabot secrets only - the rest work just fine. |
That is indeed Dependabot-specific. Still waiting on the release, unfortunately. Perhaps we can request one if it doesn't happen soon. |
@kfcampbell Any update? It's not fun to have the whole GitHub provisioning stuck because otherwise, I have to recreate all Dependabot secrets manually... |
@nikolay the library has been updated in the provider and I'll be cutting a new release later today, which should hopefully resolve the issue. Please report back here if it does not. |
Thank you very much, @kfcampbell! 🙇🏻 |
Do you mind trying v5.34.0 and reporting back? |
@kfcampbell It worked! Thank you very much! |
The code has been working for years, but suddenly, I started getting this error:
The text was updated successfully, but these errors were encountered: