diff --git a/horizon/test/settings.py b/horizon/test/settings.py index c48d74fee6..4f405af771 100644 --- a/horizon/test/settings.py +++ b/horizon/test/settings.py @@ -63,6 +63,7 @@ INSTALLED_APPS = ( ) MIDDLEWARE_CLASSES = ( + 'openstack_auth.middleware.OpenstackAuthMonkeyPatchMiddleware', 'django.middleware.common.CommonMiddleware', 'django.middleware.csrf.CsrfViewMiddleware', 'django.contrib.sessions.middleware.SessionMiddleware', diff --git a/openstack_auth/middleware.py b/openstack_auth/middleware.py new file mode 100644 index 0000000000..41bfb87412 --- /dev/null +++ b/openstack_auth/middleware.py @@ -0,0 +1,20 @@ +# Licensed under the Apache License, Version 2.0 (the "License"); you may +# not use this file except in compliance with the License. You may obtain +# a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, software +# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT +# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the +# License for the specific language governing permissions and limitations +# under the License. + +from openstack_auth import utils + +# NOTE: The main role of this middleware is to call this. +utils.patch_middleware_get_user() + + +class OpenstackAuthMonkeyPatchMiddleware(object): + pass diff --git a/openstack_auth/tests/settings.py b/openstack_auth/tests/settings.py index 93a9dcb91f..50f0c2f72e 100644 --- a/openstack_auth/tests/settings.py +++ b/openstack_auth/tests/settings.py @@ -28,6 +28,7 @@ INSTALLED_APPS = [ ] MIDDLEWARE_CLASSES = [ + 'openstack_auth.middleware.OpenstackAuthMonkeyPatchMiddleware', 'django.middleware.common.CommonMiddleware', 'django.middleware.csrf.CsrfViewMiddleware', 'django.contrib.sessions.middleware.SessionMiddleware', diff --git a/openstack_auth/tests/urls.py b/openstack_auth/tests/urls.py index b93725a2a2..0a32b08418 100644 --- a/openstack_auth/tests/urls.py +++ b/openstack_auth/tests/urls.py @@ -15,13 +15,9 @@ from django.conf.urls import include from django.conf.urls import url from django.views import generic -from openstack_auth import utils from openstack_auth import views -utils.patch_middleware_get_user() - - urlpatterns = [ url(r"", include('openstack_auth.urls')), url(r"^websso/$", views.websso, name='websso'), diff --git a/openstack_auth/urls.py b/openstack_auth/urls.py index 12d7cfb0b3..bfa95c2ac2 100644 --- a/openstack_auth/urls.py +++ b/openstack_auth/urls.py @@ -16,8 +16,6 @@ from django.conf.urls import url from openstack_auth import utils from openstack_auth import views -utils.patch_middleware_get_user() - urlpatterns = [ url(r"^login/$", views.login, name='login'), diff --git a/openstack_auth/utils.py b/openstack_auth/utils.py index 91ac31cb9f..775651f7cd 100644 --- a/openstack_auth/utils.py +++ b/openstack_auth/utils.py @@ -37,8 +37,8 @@ We need the request object to get the user, so we'll slightly modify the existing django.contrib.auth.get_user method. To do so we update the auth middleware to point to our overridden method. -Calling the "patch_middleware_get_user" method somewhere like our urls.py -file takes care of hooking it in appropriately. +Calling "patch_middleware_get_user" is done in our custom middleware at +"openstack_auth.middleware" to monkeypatch the code in before it is needed. """ diff --git a/openstack_dashboard/settings.py b/openstack_dashboard/settings.py index 697ade65ff..32cc3a4bfe 100644 --- a/openstack_dashboard/settings.py +++ b/openstack_dashboard/settings.py @@ -108,6 +108,7 @@ OPENSTACK_IMAGE_BACKEND = { } MIDDLEWARE_CLASSES = ( + 'openstack_auth.middleware.OpenstackAuthMonkeyPatchMiddleware', 'django.middleware.common.CommonMiddleware', 'django.middleware.csrf.CsrfViewMiddleware', 'django.contrib.sessions.middleware.SessionMiddleware', diff --git a/openstack_dashboard/test/helpers.py b/openstack_dashboard/test/helpers.py index 896477474f..a97abcb3dc 100644 --- a/openstack_dashboard/test/helpers.py +++ b/openstack_dashboard/test/helpers.py @@ -466,7 +466,6 @@ class APITestCase(TestCase): """ def setUp(self): super(APITestCase, self).setUp() - utils.patch_middleware_get_user() def fake_keystoneclient(request, admin=False): """Returns the stub keystoneclient.