D889: url: use native strings for header values
durin42 (Augie Fackler)
phabricator at mercurial-scm.org
Sun Oct 1 16:43:39 UTC 2017
durin42 created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D889
AFFECTED FILES
mercurial/url.py
CHANGE DETAILS
diff --git a/mercurial/url.py b/mercurial/url.py
--- a/mercurial/url.py
+++ b/mercurial/url.py
@@ -19,6 +19,7 @@
error,
httpconnection as httpconnectionmod,
keepalive,
+ pycompat,
sslutil,
util,
)
@@ -495,13 +496,13 @@
# agent string for anything, clients should be able to define whatever
# user agent they deem appropriate.
agent = 'mercurial/proto-1.0 (Mercurial %s)' % util.version()
- opener.addheaders = [('User-agent', agent)]
+ opener.addheaders = [(r'User-agent', pycompat.sysstr(agent))]
# This header should only be needed by wire protocol requests. But it has
# been sent on all requests since forever. We keep sending it for backwards
# compatibility reasons. Modern versions of the wire protocol use
# X-HgProto-<N> for advertising client support.
- opener.addheaders.append(('Accept', 'application/mercurial-0.1'))
+ opener.addheaders.append((r'Accept', r'application/mercurial-0.1'))
return opener
def open(ui, url_, data=None):
To: durin42, #hg-reviewers
Cc: mercurial-devel
More information about the Mercurial-devel
mailing list