Change in ovirt-engine-sdk[master]: sdk: Read complete HTTP reason code

[juan.hernandez at redhat.com]

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 at 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 at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: automation at ovirt.org