response of v3 OS-EC2 API doesn't contain links field

Bug #1416303 reported by wanghong
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Fix Released
Low
wanghong

Bug Description

An example of create API:

{"credential": {"access": "1d3c23ddf17b4a2dbb1702fda6f70fff", "tenant_id": "af893e3fa7e94f38bbbb0103e3a0777b", "secret": "e27ca17c32534c18a5d695f26bd766db", "user_id": "ea8e8e91c7404f9db859730df4374137", "trust_id": null}}

This is because we don't apply wrap_member or wrap_collection on the response of v3 OS-EC2 API.
https://github.com/openstack/keystone/blob/master/keystone/contrib/ec2/controllers.py#L387
https://github.com/openstack/keystone/blob/master/keystone/contrib/ec2/controllers.py#L174

Example of how collections are wrapped in other places:

https://github.com/openstack/keystone/blob/15fb5d68cd871a9d05f3bc332139e808d47af2a8/keystone/catalog/controllers.py#L308

wanghong (w-wanghong)
Changed in keystone:
assignee: nobody → wanghong (w-wanghong)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to keystone (master)

Fix proposed to branch: master
Review: https://review.openstack.org/151592

Changed in keystone:
status: New → In Progress
description: updated
Changed in keystone:
milestone: none → kilo-rc1
Changed in keystone:
importance: Undecided → Low
Changed in keystone:
milestone: kilo-rc1 → none
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to keystone (master)

Reviewed: https://review.openstack.org/151592
Committed: https://git.openstack.org/cgit/openstack/keystone/commit/?id=e6c67148feb87e5fe755d8203b62ebd41b6fa3d5
Submitter: Jenkins
Branch: master

commit e6c67148feb87e5fe755d8203b62ebd41b6fa3d5
Author: wanghong <email address hidden>
Date: Fri Jan 30 18:11:27 2015 +0800

    add missing links for v3 OS-EC2 API response

    Change-Id: Ic83109f65fc7ba077a74865a7b5c630a01203edb
    Closes-Bug: #1416303

Changed in keystone:
status: In Progress → Fix Committed
Changed in keystone:
milestone: none → kilo-rc1
Thierry Carrez (ttx)
Changed in keystone:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in keystone:
milestone: kilo-rc1 → 2015.1.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.