Change in ovirt-engine-sdk[master]: sdk: Read complete HTTP reason code
Juan Hernandez has submitted this change and it was merged.
Change subject: sdk: Read complete HTTP reason code
......................................................................
sdk: Read complete HTTP reason code
Currently the SDK doesn't read the complete HTTP reason code sent by the
server, only the first word. For example, if the server sends the
following response line:
HTTP/1.1 503 Service Unavailable
The SDK will only get the first word and then it will generate an
incomplete error report:
status: 503
reason: Service
This patch fixes that problem, so that the error report will contain the
complete HTTP reason code:
status: 503
reason: Service Unavailable
This is a regression that was introduced as part of the changes to use
PycURL instead of the builtin Python HTTP module.
Change-Id: Ic7fff3d5b5c81aa949985a085afaf67c437365c9
Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com>
---
M src/ovirtsdk/infrastructure/connectionspool.py
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Juan Hernandez: Verified; Looks good to me; Passed CI tests; Ready for merge
--
To view, visit https://gerrit.ovirt.org/42556
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic7fff3d5b5c81aa949985a085afaf67c437365c9
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine-sdk
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org