statiqdev / Statiq.Web

Statiq Web is a flexible static site generator written in .NET.
https://statiq.dev/web
Other
1.64k stars 235 forks source link

Deploying to GitHub Pages results in abuse detection mechanism kicking in #952

Closed gep13 closed 3 years ago

gep13 commented 3 years ago

When using Statiq to publish the output to GitHub pages, there are times when the following error message appears:

image

And the overall deployment fails.

Due to the number of pages that are generated in the output, and the fact that OctoKit doesn't provide any mechanism to handle the back-off required by the GitHub API, results in an overall failed build.

A suggestion has been made to use an alternative pipeline module to handle the publishing of the site in a different way, but I do feel that if Statiq ships with the ability to publish to GitHub Pages that it should handle this abuse mechanism, or at least, provide a best effort.

gep13 commented 3 years ago

@patriksvensson you mentioned that this might be something that you could provide a PR for, is that correct?

daveaglick commented 3 years ago

A good discussion and some possible mitigation strategies can be found at https://github.com/octokit/octokit.net/issues/1792. Note that Octokit will not be providing any rate limiting buffer out-of-box as discussed in that issue and it's up to callers to handle rate limiting error responses as appropriate for them.

daveaglick commented 3 years ago

I think I've got this one resolved 🤞