Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Don't add headers without value ......................................................................
sdk: Don't add headers without value Currently the SDK adds to the request all the headers passed from the generated broker classes, including those headers with no value. The result is headers that have the string "None" as the value, for example: Content-type: None Correlation-Id: None These headers, in particular the "Content-type: None" header are then rejected by the application server. This behaviour is a regression introduced by the change tht replaced the Pyhton HTTP module with PycURL, and only affects the 3.6 branck. To avoid this issue this patch changes the SDK so that it will not add the headers that don't have a value. Change-Id: I1ebbfc2197bbfdaf3765460b3ff68765065b7696 Bug-Url: https://bugzilla.redhat.com/1227655 Signed-off-by: Juan Hernandez <juan.hernan...@redhat.com> --- M src/ovirtsdk/infrastructure/connectionspool.py 1 file changed, 4 insertions(+), 3 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me; Ready for merge; Passed CI tests Jenkins CI: Verified -- To view, visit https://gerrit.ovirt.org/41880 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1ebbfc2197bbfdaf3765460b3ff68765065b7696 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernan...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com> Gerrit-Reviewer: Ravi Nori <rn...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
