<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Feb 27, 2014 at 5:28 AM, Sean Dague <span dir="ltr"><<a href="mailto:sean@dague.net" target="_blank">sean@dague.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<div class="">On 02/27/2014 08:13 AM, Doug Hellmann wrote:<br>
><br>
><br>
><br>
> On Thu, Feb 27, 2014 at 12:48 AM, Michael Davies <<a href="mailto:michael@the-davies.net">michael@the-davies.net</a><br>
</div><div class="">> <mailto:<a href="mailto:michael@the-davies.net">michael@the-davies.net</a>>> wrote:<br>
><br>
>     Hi everyone,<br>
><br>
>     Over in "Ironic Land" we're looking at removing XML support from<br>
>     ironic-api (i.e. <a href="https://bugs.launchpad.net/ironic/+bug/1271317" target="_blank">https://bugs.launchpad.net/ironic/+bug/1271317</a>)<br>
><br>
>     I've been looking, but I can't seem to find an easy way to modify<br>
>     the accepted content_types.<br>
><br>
>     Are there any wsgi / WSME / Pecan experts out there who can point me<br>
>     in the right direction?<br>
><br>
><br>
> There's no support for turning off a protocol in WSME right now, but we<br>
> could add that if we really need it.<br>
><br>
> Why would we turn it off, though? The point of dropping XML support in<br>
> some of the other projects is that they use toolkits that require extra<br>
> work to support it (either coding or maintenance of code we've written<br>
> elsewhere for OpenStack). WSME supports both protocols without the API<br>
> developer having to do any extra work.<br>
<br>
</div>Because if an interface is exported to the user, then it needs to be<br>
both Documented and Tested. So that's double the cost on the validation<br>
front, and the documentation front.<br>
<br>
Exporting an API isn't set and forget. Especially with the semantic<br>
differences between JSON and XML. And if someone doesn't feel the XML<br>
created by WSME is semantically useful enough to expose to their users,<br>
they shouldn't be forced to by the interface.<br>
<span class=""><font color="#888888"><br></font></span></blockquote><div><br></div><div>Aside from our lack of doc and test coverage for XML support and the desire to hide an untested and undocumented API (which I think are valid reasons to disable it) there is an actual technical problem.</div>

<div><br></div><div>Ironic's API relies on HTTP PATCH to modify resources, which Pecan/WSME does not abstract for us. We're using the python jsonpatch library to parse these requests. I'm not aware of a similar python library for XML support.</div>

<div><br></div><div>-Deva</div></div></div></div>