Change in ovirt-engine-sdk[master]: sdk: Decode headers using ASCII

juan.hernandez at redhat.com juan.hernandez at redhat.com
Tue Sep 1 08:06:06 UTC 2015


Juan Hernandez has submitted this change and it was merged.

Change subject: sdk: Decode headers using ASCII
......................................................................


sdk: Decode headers using ASCII

Currently the HTTP response headers are used without decoding them.
This is fine in Python 2 because strins are actually array of bytes, but
in Python 3 strings aren't array of bytes, so before using the headers
then need to be decoded using the right encoding. In this case the right
encoding is ASCII, as it is what is recommended by the HTTP protocol
specification.

Change-Id: Iee51e67addd7ba7a53c726c5ab8d2270cd348392
Related: https://bugzilla.redhat.com/1096137
Signed-off-by: Juan Hernandez <juan.hernandez at redhat.com>
---
M src/ovirtsdk/infrastructure/connectionspool.py
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Juan Hernandez: Verified; Looks good to me; Passed CI tests; Ready for merge
  Jenkins CI: Verified



-- 
To view, visit https://gerrit.ovirt.org/41331
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iee51e67addd7ba7a53c726c5ab8d2270cd348392
Gerrit-PatchSet: 7
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



More information about the Engine-commits mailing list