Extract noauth context from ContextMiddleware

Bug #992859 reported by Brian Waldon
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Glance
Fix Released
Medium
Brian Waldon

Bug Description

glance.common.context.ContextMiddleware handles coalescing keystone middleware headers into request context attributes. It also handles the situation where no upstream keystone middleware is deployed. These two things are completely opposite of one another and it makes more sense to have them be two separate middlewares. No reasonable environment will need both.

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

Fix proposed to branch: master
Review: https://review.openstack.org/7002

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

Reviewed: https://review.openstack.org/7002
Committed: http://github.com/openstack/glance/commit/eb4aaa97bd1e1c7e541a0ad109fb23dfee0f9c9d
Submitter: Jenkins
Branch: master

commit eb4aaa97bd1e1c7e541a0ad109fb23dfee0f9c9d
Author: Brian Waldon <email address hidden>
Date: Tue May 1 15:41:06 2012 -0700

    Split noauth context middleware into new class

    Use UnauthenticatedContextMiddleware in the case that you want to deploy
    without an auth service. ContextMiddleware is now solely for authenticated
    requests.

    * Rewrite ownership tests for adding/updating images through the v1 API
    * Fixes bug 992859

    Change-Id: I608671aac8300c9195769542708708afda991e04

Changed in glance:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in glance:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in glance:
milestone: folsom-1 → 2012.2
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.