Identity API spec for creating domain-config should be PUT not POST

Bug #1426448 reported by Henry Nash
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Fix Released
High
Henry Nash

Bug Description

The API for creating a domain specific config incorrectly shows

POST /domains/{domain_id}/config

instead of

PUT /domains/{domain_id}/config

...which is what the code does, since the /config part of the API is not valid until the config is create.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to keystone-specs (master)

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

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

Reviewed: https://review.openstack.org/159919
Committed: https://git.openstack.org/cgit/openstack/keystone-specs/commit/?id=a76c14732f4612986107ff54fd3f09303d8e567b
Submitter: Jenkins
Branch: master

commit a76c14732f4612986107ff54fd3f09303d8e567b
Author: Henry Nash <email address hidden>
Date: Fri Feb 27 16:41:49 2015 +0000

    Correct the use of POST for domain configs

    The API spec hints that you should use POST to create domain
    specific configs for a domain, whereas in fact you should use
    PUT, since the URL isn't technically valid until after you
    have created the config.

    Change-Id: Id17e642c95c1a627a4036f057d5761f25ed853c1
    Closes-bug: 1426448

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: kilo-3 → 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.