-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Quality of Service - Configuration #908
Comments
Any solution? |
Same issue for me. Appear to be getting the same error even with all 3 options provided. Would like to be able to only set TimeoutValue. "QoSOptions": { |
DanHarltey
added a commit
to DanHarltey/Ocelot
that referenced
this issue
Jul 1, 2020
…in a exception getting thrown. Issue ThreeMammals#908
raman-m
added
bug
Identified as a potential bug
needs validation
Issue has not been replicated or verified yet
labels
May 20, 2023
raman-m
pushed a commit
to DanHarltey/Ocelot
that referenced
this issue
May 20, 2023
…in a exception getting thrown. Issue ThreeMammals#908
raman-m
added
small effort
Likely less than a day of development effort.
accepted
Bug or feature would be accepted as a PR or is being worked on
and removed
needs validation
Issue has not been replicated or verified yet
labels
May 24, 2023
raman-m
pushed a commit
to DanHarltey/Ocelot
that referenced
this issue
Aug 2, 2023
…in a exception getting thrown. Issue ThreeMammals#908
raman-m
added a commit
that referenced
this issue
Sep 22, 2023
* Fixing an issue where only setting timeout value in QoS would result in a exception getting thrown. Issue #908 * Apply wrapping in case of two policies or more * Enable unit testing using DI pattern * Add unit tests for the PollyCircuitBreakingDelegatingHandler class * Split using sections for Ocelot.* and System.* namespaces * Fix code review issue: "more than 1" * File-scoped namespace * Fix messages * IDE1006 Naming rule violation: These words must begin with upper case characters * IDE0062 Local function can be made static * IDE1006 Naming rule violation: These words must begin with upper case characters: should_* * Remove and Sort Usings * Remove and Sort Usings * RaynaldM code review --------- Co-authored-by: Raman Maksimchuk <[email protected]>
raman-m
added
merged
Issue has been merged to dev and is waiting for the next release
and removed
accepted
Bug or feature would be accepted as a PR or is being worked on
small effort
Likely less than a day of development effort.
labels
Sep 22, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Background
TimeoutValue
in isolation of theExceptionsAllowedBeforeBreaking
andDurationOfBreak
options. I'm trying to only useTimeoutValue
property without the other two.Expected Behavior
TimeoutValue
then I should be able to call the API Gateway without any issue.Actual Behavior
TimeoutValue
returns500 Internal Server Error
. Here are logs:Additional Information
This issue can be also be reproduced with
Ocelot.AcceptanceTests
->PollyQoSTests
. If we remove propertyExceptionsAllowedBeforeBreaking
fromshould_timeout
andshould_not_timeout
methods, we getInternalServerError
. As per documentation, this should not be case.The text was updated successfully, but these errors were encountered: