D2741: wireprotoserver: remove broken optimization for non-httplib client

2018-03-09 Thread indygreg (Gregory Szorc) 
This revision was automatically updated to reflect the committed changes.
Closed by commit rHG9a6216c18ffd: wireprotoserver: remove broken optimization 
for non-httplib client (authored by indygreg, committed by ).

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D2741?vs=6747=6785

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

AFFECTED FILES
  mercurial/wireprotoserver.py

CHANGE DETAILS

diff --git a/mercurial/wireprotoserver.py b/mercurial/wireprotoserver.py
--- a/mercurial/wireprotoserver.py
+++ b/mercurial/wireprotoserver.py
@@ -328,10 +328,7 @@
 if (wsgireq.env[r'REQUEST_METHOD'] == r'POST' and
 # But not if Expect: 100-continue is being used.
 (wsgireq.env.get('HTTP_EXPECT',
- '').lower() != '100-continue') or
-# Or the non-httplib HTTP library is being advertised by
-# the client.
-wsgireq.env.get('X-HgHttp2', '')):
+ '').lower() != '100-continue')):
 wsgireq.drain()
 else:
 wsgireq.headers.append((r'Connection', r'Close'))



To: indygreg, #hg-reviewers, durin42
Cc: mercurial-devel
___
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

D2741: wireprotoserver: remove broken optimization for non-httplib client

2018-03-08 Thread indygreg (Gregory Szorc) 
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.

REVISION SUMMARY
  There was an experimental non-httplib client in core for several
  years. It was removed a week or so ago.
  
  We kept the optimization for this client in the server code. I'm
  not sure if that was intended or not. But it doesn't matter: the
  code was wrong.
  
  Because the code was accessing a WSGI environment dict, it needed to
  access the HTTP_X_HGHTTP2 key to actually read the HTTP header. So
  the code deleted by this commit wasn't actually doing anything
  meaningful. Doh.

REPOSITORY
  rHG Mercurial

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

AFFECTED FILES
  mercurial/wireprotoserver.py

CHANGE DETAILS

diff --git a/mercurial/wireprotoserver.py b/mercurial/wireprotoserver.py
--- a/mercurial/wireprotoserver.py
+++ b/mercurial/wireprotoserver.py
@@ -328,10 +328,7 @@
 if (wsgireq.env[r'REQUEST_METHOD'] == r'POST' and
 # But not if Expect: 100-continue is being used.
 (wsgireq.env.get('HTTP_EXPECT',
- '').lower() != '100-continue') or
-# Or the non-httplib HTTP library is being advertised by
-# the client.
-wsgireq.env.get('X-HgHttp2', '')):
+ '').lower() != '100-continue')):
 wsgireq.drain()
 else:
 wsgireq.headers.append((r'Connection', r'Close'))



To: indygreg, #hg-reviewers
Cc: mercurial-devel
___
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel