Patchwork D2311: keepalive: headers are native strings, mark them as such

login
register
mail settings
Submitter phabricator
Date Feb. 18, 2018, 5:42 a.m.
Message ID <differential-rev-PHID-DREV-hu6ygau3hpbzb57tg4gf-req@phab.mercurial-scm.org>
Download mbox | patch
Permalink /patch/28051/
State Superseded
Headers show

Comments

phabricator - Feb. 18, 2018, 5:42 a.m.
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/D2311

AFFECTED FILES
  mercurial/keepalive.py

CHANGE DETAILS




To: durin42, #hg-reviewers
Cc: mercurial-devel
phabricator - Feb. 18, 2018, 11:54 a.m.
pulkit added a comment.


  This one should be # skip-blame.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D2311

To: durin42, #hg-reviewers
Cc: pulkit, mercurial-devel

Patch

diff --git a/mercurial/keepalive.py b/mercurial/keepalive.py
--- a/mercurial/keepalive.py
+++ b/mercurial/keepalive.py
@@ -324,11 +324,11 @@ 
                 h.putrequest(
                     req.get_method(), urllibcompat.getselector(req),
                     **pycompat.strkwargs(skipheaders))
-                if 'content-type' not in headers:
-                    h.putheader('Content-type',
-                                'application/x-www-form-urlencoded')
-                if 'content-length' not in headers:
-                    h.putheader('Content-length', '%d' % len(data))
+                if r'content-type' not in headers:
+                    h.putheader(r'Content-type',
+                                r'application/x-www-form-urlencoded')
+                if r'content-length' not in headers:
+                    h.putheader(r'Content-length', r'%d' % len(data))
             else:
                 h.putrequest(
                     req.get_method(), urllibcompat.getselector(req),