From 1340a6a6e9ad2e57fc0d9f2b79e2eefdf044d29f Mon Sep 17 00:00:00 2001 From: Benjamin Dauvergne Date: Fri, 24 Apr 2020 13:08:09 +0200 Subject: [PATCH 6/7] misc: style (#41949) --- mellon/adapters.py | 5 +++-- mellon/views.py | 27 ++++++++++++++------------- 2 files changed, 17 insertions(+), 15 deletions(-) diff --git mellon/adapters.py mellon/adapters.py index f3d8c02..b41cdd4 100644 --- mellon/adapters.py +++ 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 mellon/views.py mellon/views.py index 179b720..bfbbcb0 100644 --- mellon/views.py +++ 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 sso_success(self, request, login): attributes = {} @@ -281,9 +282,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) -- 2.26.0