diff --git a/mellon/adapters.py b/mellon/adapters.py index 7a970a8..2b8a58f 100644 --- a/mellon/adapters.py +++ b/mellon/adapters.py @@ -374,8 +374,9 @@ class DefaultAdapter(object): continue values = saml_attributes.get(saml_attribute) if not values: - logger.warning('looking for user by saml attribute %r and user field %r, skipping because empty', - saml_attribute, user_field) + logger.warning( + 'looking for user by saml attribute %r and user field %r, skipping because empty', + saml_attribute, user_field) continue ignore_case = line.get('ignore-case', False) for value in values: diff --git a/mellon/views.py b/mellon/views.py index 9f68f30..066e752 100644 --- a/mellon/views.py +++ b/mellon/views.py @@ -202,16 +202,17 @@ class LoginView(ProfileMixin, LogMixin, View): if error_url: error_url = resolve_url(error_url) next_url = error_url or self.get_next_url(default=resolve_url(settings.LOGIN_REDIRECT_URL)) - return self.render(request, 'mellon/authentication_failed.html', - { - 'debug': settings.DEBUG, - 'reason': reason, - 'status_codes': status_codes, - 'issuer': login.remoteProviderId, - 'next_url': next_url, - 'relaystate': login.msgRelayState, - 'error_redirect_after_timeout': error_redirect_after_timeout, - }) + return self.render( + request, 'mellon/authentication_failed.html', + { + 'debug': settings.DEBUG, + 'reason': reason, + 'status_codes': status_codes, + 'issuer': login.remoteProviderId, + 'next_url': next_url, + 'relaystate': login.msgRelayState, + 'error_redirect_after_timeout': error_redirect_after_timeout, + }) def get_attribute_value(self, attribute, attribute_value): # check attribute_value contains only text @@ -289,9 +290,9 @@ class LoginView(ProfileMixin, LogMixin, View): 'saml_attributes': attributes}) else: self.log.warning('no user found for NameID %r', attributes['name_id_content']) - return self.render(request, 'mellon/user_not_found.html', { - 'saml_attributes': attributes - }) + return self.render( + request, 'mellon/user_not_found.html', + {'saml_attributes': attributes}) request.session['lasso_session_dump'] = login.session.dump() return HttpResponseRedirect(next_url)