search

ThreeMammals / Ocelot

.NET API Gateway
https://www.nuget.org/packages/Ocelot
MIT License
8.31k stars 1.63k forks source link

Add `EnableMultipleHttp2Connections` property to `HttpHandlerOptions` #2126

Open EngRajabi opened 1 month ago

EngRajabi commented 1 month ago

If you activate HTTP/2, it is better to activate the SocketsHttpHandler.EnableMultipleHttp2Connections configuration as well, which gives good results.

Docs

raman-m commented 1 month ago

Thank you, Mohsen! Is prioritization required for this PR? It seems straightforward, and we could aim to deliver it in the release following the 23.3 Hotfixes.

Currently, there is a shortage of both unit and acceptance tests to begin the review process. We need to implement a genuine HTTP/2 + The Option acceptance test to assess the stability of Ocelot Core in this scenario.

EngRajabi commented 1 month ago

It is not a high priority. I can get a load test for it. But it is time consuming