Skip to content

Make IceBox watcher report more reliable#122

Merged
zivkan merged 6 commits intomainfrom
zivkan-GithubIssueTagger-HttpRetry
Apr 30, 2025
Merged

Make IceBox watcher report more reliable#122
zivkan merged 6 commits intomainfrom
zivkan-GithubIssueTagger-HttpRetry

Conversation

@zivkan
Copy link
Copy Markdown
Member

@zivkan zivkan commented Apr 29, 2025

  • Fix IceBoxResource.resx, so it can build on Linux
  • Retry failed HTTP requests up to 5 times
  • When a failed response has a Retry-After header, respect the duration
  • Output to the console the failed HTTP response headers and body, so we can see what the failure is

Turns out the errors we were seeing were reaching github's secondary rate limit

@zivkan zivkan requested a review from a team April 29, 2025 11:32
Comment thread GithubIssueTagger/GraphQL/GitHubGraphQLClient.cs
Comment thread GithubIssueTagger/GraphQL/GitHubGraphQLClient.cs Outdated
@zivkan zivkan merged commit 5722a07 into main Apr 30, 2025
1 check passed
@zivkan zivkan deleted the zivkan-GithubIssueTagger-HttpRetry branch April 30, 2025 01:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants