durin42 accepted this revision.
durin42 added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> wireprotoserver.py:94
>      def forwardpayload(self, fp):
> -        if b'Content-Length' in self._req.headers:
> -            length = int(self._req.headers[b'Content-Length'])
> -        else:
> -            length = int(self._wsgireq.env[r'CONTENT_LENGTH'])
> +        # TODO Content-Length may not always be defined.
> +        length = int(self._req.headers[b'Content-Length'])

For our clients it always will, because we precompute the bundle to a file. 
It's gross.

REPOSITORY
  rHG Mercurial

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

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

Reply via email to