-
-
Notifications
You must be signed in to change notification settings - Fork 22.3k
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
Self hosted vercel instance gives maxDuration warning #1416
Comments
Open the vercel.json file and change the maxDuration to 10. It's not a bug, it was intentionally reduced to 30s because by default on PRO plan it sets to 1min which is too high and was causing too much memory usage. but for non-pro users the duration can only be <10s thus 30s fails. |
There were two other users having the same confusion around this issue. Maybe we should document that, after forking the repo for self-hosting they should change the duration to 10 |
@anuraghazra I think that is a good idea. I reopened the issue. Can you like the related issues? |
@anuraghazra Do you want me to update the documentation or are you doing it yourself? In the meantime allow me to create a commit that people can cherry-pick. I think it is nice to add this to the documentation. |
This commit decreases the vercel 'maxDuration' parameter to `10` such that users with the free vercel plan can also deploy the 'github-readme-stats' repository. See anuraghazra#1416 for more information.
People can cherry-pick 8af4544 to fix this issue. |
@anuraghazra Ah, maybe you can create this commit on a new branch then remove the branch. In the current form, people have to use my fork. If that is too much work we can just document the behaviour. |
No there's no need to use your fork, it will create a two level dependency (on my fork -> your fork -> user's fork). We can just document the behaviour and also improve the error msg to exactly say what users have to do to fix the issue. |
@rickstaa does the error msg shows up on the rendered card itself or in a custom vercel error page? |
@anuraghazra There were two places where the error showed up. In the vercel deployment log and as a GitHub notification since I use the github vercel app. The card itself is not displayed since the deployment fails. I temporarily broke my fork again to show the behaviour: You can view this here. |
Oh yeah then we cannot show a custom msg, I thought the error was shown in the rendered card itself that way we could parse and replace the string. but not with this then. I've added docs only. |
@anuraghazra Thanks for your pull request! I think it is clear now. |
Describe the bug
When I try to deploy the master branch on my self-hosted vercel instance (i.e. https://github.com/anuraghazra/github-readme-stats#deploy-on-your-own-vercel-instance) I get the following error:
I did some debugging and this error has been present since 81b6593.
Expected behaviour
This error should not be present.
TODO
The text was updated successfully, but these errors were encountered: