[system] Remove port number from TLS Server Name Identification (SNI). Fixes #46549...
authorSebastien Pouliot <sebastien.pouliot@gmail.com>
Wed, 14 Dec 2016 19:24:41 +0000 (14:24 -0500)
committerAlexander Köplinger <alex.koeplinger@outlook.com>
Wed, 14 Dec 2016 19:24:41 +0000 (20:24 +0100)
commitfaf2a56710b105f668bab8b71565d66297e1e340
treee1eed42d344f015aa04568b6ba12ce582d0dad0c
parent9b6a2feb2057382259b6e188f5f94ea02bc3c513
[system] Remove port number from TLS Server Name Identification (SNI). Fixes #46549 (#4120)

The port number should not be included along the host name. Otherwise
the server will refuse the connection (and we'll throw). This is a
problem when servers are not using the default (443) SSL/TLS port.

ref: https://bugzilla.xamarin.com/show_bug.cgi?id=46549

The BTLS provider was fixed but the old MonoTLS (managed) provider had
the same issue.

Another PR will be made to fix AppleTLS in xamarin-macios repo [2]

[1] https://github.com/mono/mono/pull/3939
[2] https://bugzilla.xamarin.com/show_bug.cgi?id=45994
mcs/class/System/Mono.Net.Security/MonoTlsStream.cs