Deprecate catalog replacements and whitelists

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

Bug Description

Bug #1354208 reported a security flaw in the way that we performed substitution for catalog URLs. The immediate solution was to add a whitelist of config fields that are safe to use with substitution. The long term goal is to get rid of this feature and only allow tenant_id and user_id to be used for substitution.

The first step for the Kilo release is to deprecate the feature.

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/130013

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on keystone (master)

Change abandoned by David Stanek (<email address hidden>) on branch: master
Review: https://review.openstack.org/130013
Reason: Abandoning in favor of Brant's patch: https://review.openstack.org/#/c/131007/

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-1 → 2015.1.0
Revision history for this message
huangpengtaohw (huangpengtao) wrote :

in line 896 the function _config_to_list is not working well,
the return volue (whitelisted, sensitiveto) is alwayle []

Revision history for this message
David Stanek (dstanek) wrote :

huangpengtao, Can you provide a test case that shows the bad behavior? Eye balling the code it looks OK, although I can see why it would look like the return value is alway ([], []). The reason this is not the case is that 'the_list' is used as an alias for 'whitelisted' and 'sensitive'.

If you truly think there is a bug please create a new bug with the details.

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.