# HG changeset patch
# User Gregory Szorc <gregory.szorc@gmail.com>
# Date 1462033607 25200
# Node ID 693b856a4d45567a1a0dc653540294e93a8d692d
# Parent  b962ae0a0a058d15053685dffdb508d36cd89b11
sslutil: restore old behavior not requiring a hostname argument (issue5210)

This effectively backs out changeset 1fde84d42f9c.

The http library behind ui.http2=true isn't specifying the hostname.
It is the day before the expected 3.8 release and we don't want to ship
a regression.

I'll try to restore this requirement in the 3.9 release cycle as part
of planned improvements to Mercurial's SSL/TLS interactions.

diff -r b962ae0a0a05 -r 693b856a4d45 mercurial/sslutil.py
--- a/mercurial/sslutil.py	Thu Apr 28 16:38:15 2016 -0500
+++ b/mercurial/sslutil.py	Sat Apr 30 09:26:47 2016 -0700
@@ -120,9 +120,6 @@
       server (and client) support SNI, this tells the server which certificate
       to use.
     """
-    if not serverhostname:
-        raise error.Abort('serverhostname argument required')
-
     # Despite its name, PROTOCOL_SSLv23 selects the highest protocol
     # that both ends support, including TLS protocols. On legacy stacks,
     # the highest it likely goes in TLS 1.0. On modern stacks, it can