-
Notifications
You must be signed in to change notification settings - Fork 894
Open
Labels
Type: DocumentationImprovements or additions to documentationImprovements or additions to documentationhelp wantedWe will welcome a contributionWe will welcome a contribution
Milestone
Description
In the current docs about YARP Timeouts, there’s no mention of ActivityTimeout.
This is confusing because ActivityTimeout
has a default of 100 seconds, which silently overrides route-level Timeout
values longer than that.
For example:
- If you configure a route
Timeout
of 300 seconds, it will be ignored because the request is still canceled after the defaultActivityTimeout
of 100s. - But if you configure a route
Timeout
of 1 second, it does take effect.
This behavior can make debugging really painful, since developers may think their configured Timeout
is being honored when in reality it’s capped by ActivityTimeout
, it drove us insane this morning :)
The docs should explicitly call out the interaction between Timeout
and ActivityTimeout
, and clarify which one takes precedence in different scenarios.
LucioPelinson and Xulei-NL
Metadata
Metadata
Assignees
Labels
Type: DocumentationImprovements or additions to documentationImprovements or additions to documentationhelp wantedWe will welcome a contributionWe will welcome a contribution