comparison mercurial/httppeer.py @ 30564:07bcd1bf6151

httppeer: assign Vary request header last

In preparation for adding another value to it in a subsequent patch.

While I was here, I added some empty lines because walls of text
are hard to read.

30563:e118233172fe

        headers = args.pop('headers', {})

        self.ui.debug("sending %s command\n" % cmd)
        q = [('cmd', cmd)]
        headersize = 0
        # Important: don't use self.capable() here or else you end up
        # with infinite recursion when trying to look up capabilities
        # for the first time.
        postargsok = self.caps is not None and 'httppostargs' in self.caps
        # TODO: support for httppostargs when data is a file-like

                # The headers can typically carry more data than the URL.
                encargs = urlreq.urlencode(sorted(args.items()))
                headerfmt = 'X-HgArg-%s'
                contentlen = headersize - len(headerfmt % '000' + ': \r\n')
                headernum = 0
                varyheaders = []
                for i in xrange(0, len(encargs), contentlen):
                    headernum += 1
                    header = headerfmt % str(headernum)
                    headers[header] = encargs[i:i + contentlen]
                    varyheaders.append(header)
                headers['Vary'] = ','.join(varyheaders)
            else:
                q += sorted(args.items())
        qs = '?%s' % urlreq.urlencode(q)
        cu = "%s%s" % (self._url, qs)
        size = 0

        if size and self.ui.configbool('ui', 'usehttp2', False):
            headers['Expect'] = '100-Continue'
            headers['X-HgHttp2'] = '1'
        if data is not None and 'Content-Type' not in headers:
            headers['Content-Type'] = 'application/mercurial-0.1'
        req = self.requestbuilder(cu, data, headers)
        if data is not None:
            self.ui.debug("sending %s bytes\n" % size)
            req.add_unredirected_header('Content-Length', '%d' % size)
        try:
            resp = self.urlopener.open(req)

30564:07bcd1bf6151

        headers = args.pop('headers', {})

        self.ui.debug("sending %s command\n" % cmd)
        q = [('cmd', cmd)]
        headersize = 0
        varyheaders = []
        # Important: don't use self.capable() here or else you end up
        # with infinite recursion when trying to look up capabilities
        # for the first time.
        postargsok = self.caps is not None and 'httppostargs' in self.caps
        # TODO: support for httppostargs when data is a file-like

                # The headers can typically carry more data than the URL.
                encargs = urlreq.urlencode(sorted(args.items()))
                headerfmt = 'X-HgArg-%s'
                contentlen = headersize - len(headerfmt % '000' + ': \r\n')
                headernum = 0
                for i in xrange(0, len(encargs), contentlen):
                    headernum += 1
                    header = headerfmt % str(headernum)
                    headers[header] = encargs[i:i + contentlen]
                    varyheaders.append(header)
            else:
                q += sorted(args.items())
        qs = '?%s' % urlreq.urlencode(q)
        cu = "%s%s" % (self._url, qs)
        size = 0

        if size and self.ui.configbool('ui', 'usehttp2', False):
            headers['Expect'] = '100-Continue'
            headers['X-HgHttp2'] = '1'
        if data is not None and 'Content-Type' not in headers:
            headers['Content-Type'] = 'application/mercurial-0.1'

        headers['Vary'] = ','.join(varyheaders)
        req = self.requestbuilder(cu, data, headers)

        if data is not None:
            self.ui.debug("sending %s bytes\n" % size)
            req.add_unredirected_header('Content-Length', '%d' % size)
        try:
            resp = self.urlopener.open(req)