Comment 5 for bug 1189121

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

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

commit d0d4976f284361375ad8dc34d0817fbdae900ddf
Author: Wu Wenxiang <wu.wenxiang@99cloud.net>
Date: Sun Jun 9 18:10:30 2013 +0800

    Correct the resolving api logic in stat middleware

    If the admin_port string is a substring of public_port string, all the
    requests would be stated as 'admin' either their real dest port is
    admin_port or public_port. It's due to the incorrect logic in stat
    middleware.
    For example, if public_port = 35000, admin_port = 5000, the first
    judgement branch: "if str(CONF.admin_port) in host" in
    StatsMiddleware::_resolve_api() would always return "True" either the
    host port number equal to 5000 or 35000, so that the following judgement
    branches would be incorrectly ignored.

    Fixes bug #1189121
    Change-Id: I1086b7d11f83dd218d66376f79747a1f720eb807