-
Notifications
You must be signed in to change notification settings - Fork 361
[Feature Request] Use HTTP 2.0 protocol #5209
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
Comments
@MZOLN - any thoughts on this? |
I had a look at this and it looks like the default version is 2 for programs written in .NET and 1.1 for .NET FWK. As such, it seems that the better option would be to migrate the app to .NET |
It is 2.0 only for netcore 2.2 and 2.1. Otherwise it is still 1.1. For example in net8 it is still 1.1. I think it is up to the library author to decide. BTW Google API client has the same issue. You can say that it’s probably not that important to have 2.0 when getting a token as you won’t do it in parallel but that depends. |
I could probably intercept it in HttpClient and modify to become 2.0 as a workaround but overall of endpoint supports it why 1.1 ? |
MSAL needs to talk to other identity providers, for example ADFS (on-prem AD) and 2.0 might not work. |
MSAL client type
Confidential
Problem statement
HttpManager internally creates
HttpRequestMessage
which by default usesHTTP 1.1
Ideally should use
HTTP 2.0
Even though my factory makes
HTTP 2.0
default it won't workProposed solution
Alternatives
Keep using
HTTP 1.1
The text was updated successfully, but these errors were encountered: