Comment 3 for bug 854104

Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Fix merged to keystone (master)

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

 status fixcommitted
 done

commit eaf3b929f53572bfceb9271cc5496493708aa0cb
Author: Yogeshwar Srikrishnan <email address hidden>
Date: Fri Nov 11 02:35:32 2011 -0600

    Bug #854104
      - Changes to allow admin url to be shown only for admin users.
      - Additional test asserts to verify.

    This was implemented by adding a parameter to the AuthData __init__
    (the class that renders auth call responses). The url_types parameter
    filters which types of URLs to return; admin, intenrnal, public.
    And in logic/service.py, there is authorization logic which checks
    the caller's rights and based on that determines if they can see
    the 'admin' URL. If not, it takes it out of the array passed to
    the url_types paramater passed to the AuthData initializer.

    Added method documentation and renamed a couple in service.py in
    the process to improve code understandability. More needs to be
    done, but moving the ball forward.

    Change-Id: I3fb90c31617da7647a211901d647cfb53f182be2