OpenStack Identity (keystone)

Service API :5000/v2.0/tenants doesn't check HTTP method

Bug #1006793 reported by Jason Xu
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Fix Released
Low
Dolph Mathews
OpenStack Identity (keystone) 2012.2 "folsom"

Bug Description

Service API :5000/v2.0/tenants doesn't check HTTP method

This API requires GET method, but PUT,POST,DELETE can get the same result.

While Admin API :35357/v2.0/tenants is ok, it does check the HTTP method, improper method will get a 404 error.

Dolph Mathews (dolph)
Changed in keystone:
importance: Undecided → Low
status: New → Triaged
Revision history for this message
Dolph Mathews (dolph) wrote :

Confirmed; give a valid token, I was able to successfully list the available tenants using GET, POST, PUT, and DELETE without consequence: http://paste.openstack.org/raw/18324/

Changed in keystone:
assignee: nobody → Dolph Mathews (dolph)
milestone: none → folsom-2
status: Triaged → Confirmed
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/8107

Changed in keystone:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to keystone (master)

Reviewed: https://review.openstack.org/8107
Committed: http://github.com/openstack/keystone/commit/a68d530133b9687f85dad4edf74d4b69a71e4959
Submitter: Jenkins
Branch: master

commit a68d530133b9687f85dad4edf74d4b69a71e4959
Author: Dolph Mathews <email address hidden>
Date: Sun Jun 3 12:36:27 2012 -0600

    Fixed typo in routing conditions (bug 1006793)

    The kwarg should be 'method' not 'methods', per Routes docs.
    See: http://routes.readthedocs.org/en/latest/setting_up.html#conditions

    Example in lieu of negative tests...
    Without this change: http://paste.openstack.org/raw/18326/
    With this change: http://paste.openstack.org/raw/18325/

    Change-Id: If91fc0c79b320652674c68c433989a60c098dc1e

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

Other bug subscribers

Remote bug watches

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