Skip to content
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

Unable to use Quality of Service #1560

Closed
somasundar1997 opened this issue Jan 28, 2022 · 9 comments
Closed

Unable to use Quality of Service #1560

somasundar1997 opened this issue Jan 28, 2022 · 9 comments

Comments

@somasundar1997
Copy link

somasundar1997 commented Jan 28, 2022

The enumerable of policies to form the wrap must contain at least two policies

Error Code: UnableToCompleteRequestError Message: Error making http request, exception: System.ArgumentException: The enumerable of policies to form the wrap must contain at least two policies. (Parameter 'policies')
at Polly.Policy.WrapAsync(IAsyncPolicy[] policies)
at Ocelot.Provider.Polly.PollyCircuitBreakingDelegatingHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken) at System.Net.Http.HttpClient.g__Core|83_0(HttpRequestMessage request, HttpCompletionOption completionOption, CancellationTokenSource cts, Boolean disposeCts, CancellationTokenSource pendingRequestsCts, CancellationToken originalCancellationToken)
at Ocelot.Requester.HttpClientHttpRequester.GetResponse(HttpContext httpContext) errors found in ResponderMiddleware. Setting error response for request path:/GetToken, request method: POST
info: Microsoft.AspNetCore.Hosting.Diagnostics[2]

Specifications

  • Version: 18.0.0 (both Ocelot & Ocelot.Provider.Polly)
  • Dotnet Version: 6.0

Repository

I have attached the repository Url for your reference.
Repository Url: https://github.com/somasundar1997/TestGateway

@carmendiem
Copy link

did you ever resolve this? if so how? thanks

@somasundar1997
Copy link
Author

@carmendiem I think this issue already has a PR #1279 I found that a few days after creating the issue. I was using this solution from then upto now that is working fine.

@carmendiem
Copy link

@somasundar1997 can you clarify how you fixed the issue? it doesn't seem like that PR was merged. thanks again :)

@somasundar1997
Copy link
Author

@carmendiem I used that branch's build files(dll) in my project to fix the issue.

@somasundar1997
Copy link
Author

@carmendiem when can I expect this PR #1279 to get merged

@somasundar1997
Copy link
Author

@carmendiem ???

@carmendiem
Copy link

I don't know, I was only asking previously because I encountered the same issue in my code.

@raman-m
Copy link
Member

raman-m commented Jan 4, 2024

@somasundar1997 and @carmendiem
Hi! Thanks for discussion in the past!

PR #1279 was merged on Sep 22, 2023 in commit 21bc4a4 and released on Oct 7, 2023.

Going to close this issue...
I look forward to your future contributions with interesting PRs! 😉
Stay with Ocelot! 🐅

@raman-m
Copy link
Member

raman-m commented Jan 4, 2024

Already fixed by PR #1279

@somasundar1997 FYI
We have some interesting updates of QoS in version 22.0.0. See commit dabb4b5 plz!

@raman-m raman-m closed this as completed Jan 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants