diff options
author | Jenkins <jenkins@review.openstack.org> | 2015-07-20 16:22:44 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2015-07-20 16:22:44 +0000 |
commit | cbb7218357df0573d320462c838712b0659a0dfd (patch) | |
tree | 9e8018e6f902df89969eee2c996da46cb29de4e4 | |
parent | bb311fac5acfb088048790a40f0e70ecad1577a2 (diff) | |
parent | 3f1eb3f3c6e3fbf8bff010d76362558f7e22b721 (diff) | |
download | django_openstack_auth-cbb7218357df0573d320462c838712b0659a0dfd.tar.gz |
Merge "Prepend WEBROOT to redirect URL for WebSSO" into stable/kilo
-rw-r--r-- | openstack_auth/utils.py | 9 | ||||
-rw-r--r-- | openstack_auth/views.py | 2 |
2 files changed, 10 insertions, 1 deletions
diff --git a/openstack_auth/utils.py b/openstack_auth/utils.py index d861010..358f772 100644 --- a/openstack_auth/utils.py +++ b/openstack_auth/utils.py @@ -183,6 +183,15 @@ def is_websso_enabled(): return websso_enabled and keystonev3_plus +def build_absolute_uri(request, relative_url): + """Ensure absolute_uri are relative to WEBROOT.""" + webroot = getattr(settings, 'WEBROOT', '') + if webroot.endswith("/") and relative_url.startswith("/"): + webroot = webroot[:-1] + + return request.build_absolute_uri(webroot + relative_url) + + def has_in_url_path(url, sub): """Test if the `sub` string is in the `url` path.""" scheme, netloc, path, query, fragment = urlparse.urlsplit(url) diff --git a/openstack_auth/views.py b/openstack_auth/views.py index 8d5a0f5..68a1b08 100644 --- a/openstack_auth/views.py +++ b/openstack_auth/views.py @@ -60,7 +60,7 @@ def login(request, template_name=None, extra_context=None, **kwargs): protocol = request.POST.get('auth_type', 'credentials') if utils.is_websso_enabled() and protocol != 'credentials': region = request.POST.get('region') - origin = request.build_absolute_uri('/auth/websso/') + origin = utils.build_absolute_uri(request, '/auth/websso/') url = ('%s/auth/OS-FEDERATION/websso/%s?origin=%s' % (region, protocol, origin)) return shortcuts.redirect(url) |