Handle XML as well as JSON in ReST API

Bug #1072906 reported by Angus Salkeld
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Heat
Grizzly
Invalid
Medium
Unassigned

Bug Description

Handle XML as well as JSON in ReST API

Revision history for this message
Angus Salkeld (asalkeld) wrote :

http://blog.doughellmann.com/2012/10/ods-grizzly-choosing-wsgi-framework-for.html

I think Doug was pushing "Pecan", the reason I suggest this is we would get xml for "free".

(added Doug to subscribers, maybe he can chime in).

Changed in heat:
status: New → Triaged
importance: Undecided → Medium
Revision history for this message
Doug Hellmann (doug-hellmann) wrote :

We discussed using Pecan (http://pecan.readthedocs.org/en/latest/) and WSME (http://pypi.python.org/pypi/WSME) together. I'm supposed to rebuild the ceilometer API server as an example.

Steven Dake (sdake)
Changed in heat:
milestone: none → grizzly-3
Revision history for this message
Steven Hardy (shardy) wrote :

Although I don't doubt there may be advantages in using Pecan, we don't need to do that in order to get XML support in the ReST API (IMO), we already have wsgi middleware which supports serializing/deserializing both XML and JSON for the CFN API, so I don't see why the same (or very similar) code won't work for the ReST API?

See heat/common/wsgi.py - the main change required AFAICS is to properly handle the content-type header in the request, since AWS does a weird thing using a "ContentType" query parameter instead.

Revision history for this message
Doug Hellmann (doug-hellmann) wrote :

The main work to implement this is done in the v2 API.

Changed in heat:
status: Triaged → Fix Committed
status: Fix Committed → Confirmed
Revision history for this message
Doug Hellmann (doug-hellmann) wrote :

Oops, sorry, I thought this was a ceilometer ticket.

Revision history for this message
Zane Bitter (zaneb) wrote :

Replaced with https://blueprints.launchpad.net/heat/+spec/rest-xml since this is actually a feature request, not a bug.

Changed in heat:
status: Confirmed → Invalid
Steven Dake (sdake)
no longer affects: heat
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers