Change the default version discovery URLs

Bug #1297083 reported by OpenStack Infra
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openstack-manuals
Fix Released
High
Tom Fifield

Bug Description

https://review.openstack.org/78068
commit 7a760caa5d26e87879fe7e0276a5603c697cc717
Author: Jamie Lennox <email address hidden>
Date: Wed Mar 5 12:06:30 2014 +1000

    Change the default version discovery URLs

    The default discovery URLs for when the admin_endpoint and
    public_endpoint configuration values are unset is to point to the
    localhost. This is wrong in all but the most trivial cases.

    It also has the problem of not being able to distinguish for the public
    service whether it was accessed via the 'public' or 'private' endpoint,
    meaning that all clients that correctly do discovery will end up routing
    to the public URL.

    The most sensible default is to simply use the requested URL as the
    basis for pointing to the versioned endpoints as it at least assumes
    that the endpoint is accessible relative to the location used to arrive
    on the page.

    As mentioned in comments this is not a perfect solution. HOST_URL is the
    URL not including path (ie http://server:port) so we do not have access
    to the prefix automatically. Unfortunately the way keystone uses these
    endpoints I don't see a way of improving that without a more substantial
    redesign.

    This patch is ugly because our layers are so intertwined. It should be
    nicer with pecan.

    DocImpact: Changes the default values of admin_endpoint and
    public_endpoint and how they are used. In most situations now these
    values should be ignored in configuration.

    Change-Id: Ia6d9fbeb60ada661dc2052c9bd51db7a1dc8cd4b
    Closes-Bug: #1288009

Tags: keystone
Tom Fifield (fifieldt)
Changed in openstack-manuals:
status: New → Confirmed
importance: Undecided → High
milestone: none → icehouse
Revision history for this message
Tom Fifield (fifieldt) wrote :

This will be picked up in the next auto-config run.

Revision history for this message
Tom Fifield (fifieldt) wrote :

confirmed in config-ref

Changed in openstack-manuals:
status: Confirmed → Fix Released
assignee: nobody → Tom Fifield (fifieldt)
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.