GNOME Bugzilla – Bug 773726
/api should be replaced with /ovirt-engine/api
Last modified: 2016-11-25 17:58:02 UTC
Since oVirt 3.4, /ovirt-engine/api is preffered, and /api is obsoleted [1]. Since 4.0, /api is removed [2]. Tried to patch [3], built on Fedora 24, but it still did not work, with: (gnome-boxes:29697): libgovirt-WARNING **: Passing a full http:// or https:// URI to ovirt_proxy_new() is deprecated (gnome-boxes:29697): libgovirt-WARNING **: Passing an URI ending in /api to ovirt_proxy_new() is deprecated Not sure why. Perhaps some other places need to be patched. [1] https://bugzilla.redhat.com/961677 [2] https://gerrit.ovirt.org/44420 [3] https://github.com/didib/gnome-boxes/commit/f846bfcb7a98b206c6e7341026c046cd1cb222fc
Created attachment 340591 [details] [review] virt-broker: Do not sanitize oVirt URI Since libgovirt commit 086bd88, the proxy URI no longer needs to have the "/api" path neither the https prefix. The sanitization is now handled by libgovirt.
Created attachment 340592 [details] [review] ovirt-broker: Do not sanitize oVirt URI Since libgovirt commit 086bd88, the proxy URI no longer needs to have the "/api" path neither the https prefix. The sanitization is now handled by libgovirt.
(In reply to Felipe Borges from comment #1) > Created attachment 340591 [details] [review] [review] > virt-broker: Do not sanitize oVirt URI > > Since libgovirt commit 086bd88, the proxy URI no longer needs to > have the "/api" path neither the https prefix. The sanitization > is now handled by libgovirt. Cool. Please update the libgovirt required version first, in a separate patch.
Created attachment 340759 [details] [review] configure: Bump libgovirt minimum version
Created attachment 340771 [details] [review] configure: Bump libgovirt minimum version
Attachment 340592 [details] pushed as 95ee507 - ovirt-broker: Do not sanitize oVirt URI