summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJenkins <jenkins@review.openstack.org>2015-07-20 16:22:44 +0000
committerGerrit Code Review <review@openstack.org>2015-07-20 16:22:44 +0000
commitcbb7218357df0573d320462c838712b0659a0dfd (patch)
tree9e8018e6f902df89969eee2c996da46cb29de4e4
parentbb311fac5acfb088048790a40f0e70ecad1577a2 (diff)
parent3f1eb3f3c6e3fbf8bff010d76362558f7e22b721 (diff)
downloaddjango_openstack_auth-cbb7218357df0573d320462c838712b0659a0dfd.tar.gz
Merge "Prepend WEBROOT to redirect URL for WebSSO" into stable/kilo
-rw-r--r--openstack_auth/utils.py9
-rw-r--r--openstack_auth/views.py2
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)