From 9b6b559eca9822fa887f7cac6b051be126c1d22f Mon Sep 17 00:00:00 2001 From: Ivan Kolodyazhny Date: Thu, 21 Jun 2018 11:19:33 +0300 Subject: [PATCH] Do not redirect to the /identity tab admin users on login. The /identity url may have more resources than project page so it loads slowly. We can improve user experience by setting different login url. Change-Id: I6ce91974d101d80e9685826dc4e6a6495e6a5da3 Closes-Bug: #1778006 --- openstack_dashboard/views.py | 17 +---------------- ...dmin-redirect-on-login-6a818fe886621a14.yaml | 6 ++++++ 2 files changed, 7 insertions(+), 16 deletions(-) create mode 100644 releasenotes/notes/admin-redirect-on-login-6a818fe886621a14.yaml diff --git a/openstack_dashboard/views.py b/openstack_dashboard/views.py index 60ac9b31c2..aee81a442f 100644 --- a/openstack_dashboard/views.py +++ b/openstack_dashboard/views.py @@ -25,7 +25,6 @@ from django.views.generic import TemplateView from six.moves import urllib import horizon -from horizon import base from horizon import exceptions from horizon import notifications @@ -36,21 +35,7 @@ MESSAGES_PATH = getattr(settings, 'MESSAGES_PATH', None) def get_user_home(user): - try: - token = user.token - except AttributeError: - raise exceptions.NotAuthenticated() - # Domain Admin, Project Admin will default to identity - dashboard = None - if token.project.get('id') is None or user.is_superuser: - try: - dashboard = horizon.get_dashboard('identity') - except base.NotRegistered: - pass - - if dashboard is None: - dashboard = horizon.get_default_dashboard() - + dashboard = horizon.get_default_dashboard() return dashboard.get_absolute_url() diff --git a/releasenotes/notes/admin-redirect-on-login-6a818fe886621a14.yaml b/releasenotes/notes/admin-redirect-on-login-6a818fe886621a14.yaml new file mode 100644 index 0000000000..e758b26e1c --- /dev/null +++ b/releasenotes/notes/admin-redirect-on-login-6a818fe886621a14.yaml @@ -0,0 +1,6 @@ +--- +fixes: + - | + Do not redirect to the /identity tab admin users on login. Now `user_home` + config options works in the same way for all users. + [`bug/1778006 `_]