The version in HttpRequestMessage had no validate

[WeihanLi]

When I try to specify the HTTP version for the HTTP request, I found that there's no validation on the version, you can set whatever version you want, the request message ToString() would output the version directly

For example:

var request = new HttpRequestMessage(HttpMethod.Get, "https://google.com")
{
    Version = new Version(4, 0)
};
Console.WriteLine(request.ToString());

output:

Method: GET, RequestUri: 'https://google.com/', Version: 4.0, Content: <null>, Headers:
{
}

while the HTTP version used is HTTP/1.1, not the 4.0

Maybe the request version would be misleading if the HTTP version in HttpRequestMessage is not the same with the actual
HTTP version

[ghost]

Tagging subscribers to this area: @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

Issue Details

---

When I try to specify the HTTP version for the HTTP request, I found that there's no validation on the version, you can set whatever version you want, the request message ToString() would output the version directly

For example:

var request = new HttpRequestMessage(HttpMethod.Get, "https://google.com")
{
    Version = new Version(4, 0)
};
Console.WriteLine(request.ToString());

output:

Method: GET, RequestUri: 'https://google.com/', Version: 4.0, Content: <null>, Headers:
{
}

while the HTTP version used is HTTP/1.1, not the 4.0

Maybe the request version would be misleading if the HTTP version in HttpRequestMessage is not the same with the actual
HTTP version

Author:	WeihanLi
Assignees:	-
Labels:	area-System.Net.Http, untriaged
Milestone:	-

[MihaZupan]

The default HttpVersionPolicy is RequestVersionOrLower, so establishing 1.1 here is fine.

If you instead set it to Exact or RequestVersionOrHigher, you should see an exception being thrown before anything is sent.

If you want to see the actual version that was established, you have to look at the HttpResponseMessage

[karelz]

If you want to avoid typos, you can use pre-defined versions: https://docs.microsoft.com/en-us/dotnet/api/system.net.httpversion

Closing as By Design.

[WeihanLi]

The request message version is the same as the request message version in the response message, while the HTTP version in the response message is 1.1, and it seemed it would not throw an exception even when I set the DefaultVersionPolicy = HttpVersionPolicy.RequestVersionExact for httpClient

When I set DefaultVersionPolicy = HttpVersionPolicy.RequestVersionExact for the request message, I would get an exception

Thanks for your quick answer @MihaZupan @Kahbazi

[MihaZupan]

That's correct, the version policy set on HttpClient applies to helper methods that take Uri/string.
If you provide an HttpRequestMessage yourself, its policy applies.

[WeihanLi]

Got it, great thanks @MihaZupan