Change in vdsm[master]: xmlrpc: Support HTTP 1.1

asegurap at redhat.com asegurap at redhat.com
Mon Feb 10 23:46:35 UTC 2014


Antoni Segura Puimedon has posted comments on this change.

Change subject: xmlrpc: Support HTTP 1.1
......................................................................


Patch Set 2:

(1 comment)

Only a first read. Have to check the Python/Lib code

http://gerrit.ovirt.org/#/c/24294/2/lib/vdsm/utils.py
File lib/vdsm/utils.py:

Line 161:     # Additionally, add "Content-Length: 0" header on internal errors, when we
Line 162:     # don't send any content. This is required by HTTP 1.1, otherwise the
Line 163:     # client does not have any clue that the response was finished.
Line 164:     #
Line 165:     # These changes were taken from Python.7 version of this class. If we are
s/Python.7/Python2.7/
Line 166:     # running on Python 2.7, these changes are not needed, hence we override
Line 167:     # the methods only on Python 2.6.
Line 168: 
Line 169:     if sys.version_info[:2] == (2, 6):


-- 
To view, visit http://gerrit.ovirt.org/24294
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie033d5c53c81c8db99d5c26697a1727be030e0b4
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list