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
Runtime version getting timeout #650
Runtime version getting timeout #650
Comments
I haven't heard of any google cloud outages. Did this happen in prod? But we definitely should have some kind of fallback. This is also a DevX problem as development is hindered while offline. My suggestion would be:
|
@sebastianbenz this happen in production. |
@sebastianbenz also we can't controll this behavior see https://github.com/zeit/next.js/blob/2ba352da39ee00b6595aecdc9ffb2f103e803a85/packages/next/next-server/server/optimize-amp.ts#L9 |
Thanks for the details. This was most likely an issue in your prod environment. The runtime version is served from the same endpoint as the AMP runtime which had no outages. Nevertheless, we should fix optimizer falling because of this. Sidenote: it's going to be possible to configure optimizer in Next.js once vercel/next.js#10705 has landed in prod. |
@sebastianbenz thanks again maybe my server had some issue, either way following next pull request |
I'm getting the same issue when using the optimizer to build from my dev machine... FetchError: request to https://cdn.ampproject.org/rtv/metadata failed, reason: read ECONNRESET |
I'm having the same issue in production from time to time with my NextJS App, unfortunately this results in an Internal server error even if the user is not navigating an AMP page. Anyone found a workaround or a solution for this? |
This PR extracts fetching and configuration of runtime parameters into a separate function outside the transformers. This externalizes and centralizes runtime parameter handling enabling, removing the responsibility of handling runtime configuration inside transformers (with the potential for duplication) and enabling future optimizations of file system based caching of runtime artifacts (see #650).
This PR extracts fetching and configuration of runtime parameters into a separate function outside the transformers. This externalizes and centralizes runtime parameter handling enabling, removing the responsibility of handling runtime configuration inside transformers (with the potential for duplication) and enabling future optimizations of file system based caching of runtime artifacts (see #650).
* pass validator rules object via params * Extract RuntimeParameters This PR extracts fetching and configuration of runtime parameters into a separate function outside the transformers. This externalizes and centralizes runtime parameter handling enabling, removing the responsibility of handling runtime configuration inside transformers (with the potential for duplication) and enabling future optimizations of file system based caching of runtime artifacts (see #650). * update to amp.dev * use 15 digit version numbers * update jsdoc
Yeah, I'm also seeing this issue in production. Curious if anyone found a workaround to the 502 in production |
I fixed it by porting over all my code to a new nextjs project and redeploying it. Sucks, but it seemed to have worked. |
The fix is already in the latest Next.js canary. Would be great if someone could verify this. |
I'm not using Next.js but had the same error in my server using amp-optimizer-express. The error seems to be fixed by #747. Thanks for opening this issue and fixing it. |
any update on how to solve this problem? |
my nextjs application using
optimizer
but last 24 hours getting runtime version timeout error, also this happen before 3-4 months ago maybe google cloud outages happen latelty.so we need some fallback version for runtime-version
https://github.com/ampproject/amp-toolbox/blob/master/packages/runtime-version/lib/RuntimeVersion.js#L76-L86
update:
The text was updated successfully, but these errors were encountered: