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
YARP not able to detect UseRequestTimeouts middleware #2404
Comments
Does TimeoutPolicy work instead? |
I am getting same error with TimeoutPolicy as well.
|
@jasidhu Is by any chance the debugger is attached? The timeout middleware skips if debugger is attached and there will be no |
We are having the same issue which is keeping us from upgrading to .NET 8.0. Below is a snippet of our .NET Core MVC app's Program.cs.
--Route in appsettings.json throwing error. |
As far as I can tell we will currently incorrectly flag requests in the following cases:
Given a config with just a timeout set (e.g. |
Triage: We should improve the check here to at least avoid throwing when the debugger is attached. |
@MihaZupan Yes. It is failing only when debugger is attached. |
Thanks for running this down. We've been instructed internally to wait on our upgrade to .NET 8.0 until this issue in YARP is resolved. |
Is there any way to work on .NET 8, before the next version? |
As a workaround, you could disable request timeouts in YARP and rely on the activity timeout instead |
Is there a release schedule for when the timeout bugs will be resolved in YARP? In our application we're having timeout issues with legacy report embedding and would also like to be able to upgrade to .NET 8 sooner rather than later. |
My understanding is that request timeouts are an opt-in (off by default) .NET 8 feature, where you can opt-in via the YARP config. |
Describe the bug
I am trying to use .net8 request timeout middleware with Yarp proxy 2.1.0 but proxy is throwing following error.
To Reproduce
Error Message: The timeout was not applied for route 'route1', ensure
IApplicationBuilder.UseRequestTimeouts()
is called betweenIApplicationBuilder.UseRouting()
andIApplicationBuilder.UseEndpoints()
Further technical details
The config
Pipeline:
The text was updated successfully, but these errors were encountered: