Parsing of service catalog should be less error prone
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Dashboard (Horizon) |
Fix Released
|
High
|
Lin Hua Cheng | ||
django-openstack-auth |
Fix Released
|
High
|
Lin Hua Cheng |
Bug Description
Currently, parsing of the service catalog is hard-coded and dependent on the data structure of the service catalog.
For example:
in user.py
@property
def available_
"""Returns list of unique region name values in service catalog."""
regions = []
if self.service_
for service in self.service_
if service['type'] == 'identity':
for endpoint in service[
return regions
This code is prone to issue if the structure of the service catalog changes, it should be using the public interfaces of Service Catalog object from KSC when parsing rather than directly accessing the service_catalog dictionary.
Changed in horizon: | |
milestone: | kilo-3 → kilo-rc1 |
Changed in horizon: | |
status: | Fix Committed → Fix Released |
Changed in horizon: | |
milestone: | kilo-rc1 → 2015.1.0 |
Changed in django-openstack-auth: | |
status: | Fix Committed → Fix Released |
Even better would be to use the auth plugins and let them handle the service catalog for you.