Skip to content
Snippets Groups Projects
Commit c3d416d1 authored by Art's avatar Art :lizard:
Browse files

Prevent onelogin toolkit breaking from lazy url resolvers.

parent daf11a8c
No related branches found
No related tags found
No related merge requests found
......@@ -63,15 +63,19 @@ class LogInView(SAMLMixin, View):
def get_next_url(request):
next_url = request.GET.get(REDIRECT_FIELD_NAME, None) or conf.settings.LOGIN_REDIRECT_URL or "/"
logger.debug("Will ask IDP to redirect after login to: {}".format(next_url))
return next_url
return str(next_url)
class LogOutView(View):
def get(self, request, *args, **kwargs):
contrib_auth.logout(request)
return http.HttpResponseRedirect(self.get_next_url(request))
@staticmethod
def get_next_url(request):
next_url = request.GET.get(REDIRECT_FIELD_NAME, None) or conf.settings.LOGOUT_REDIRECT_URL or "/"
return http.HttpResponseRedirect(next_url)
return str(next_url)
@method_decorator(never_cache, "dispatch")
......@@ -101,7 +105,7 @@ class ACSAuthNView(SAMLMixin, View):
logger.warning("Did not receive RelayState (redirect target) from the IDP.")
next_url = conf.settings.LOGOUT_REDIRECT_URL or "/"
logger.debug("From ACS redirecting to {}".format(next_url))
return next_url
return str(next_url)
def log_in_user(self, request, auth):
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment