diff --git a/openstack_auth/forms.py b/openstack_auth/forms.py index 107fb191c5..a7baeeae8e 100644 --- a/openstack_auth/forms.py +++ b/openstack_auth/forms.py @@ -136,19 +136,15 @@ class Login(django_auth_forms.AuthenticationForm): password=password, user_domain_name=domain, auth_url=region) - msg = 'Login successful for user "%(username)s", remote address '\ - '%(remote_ip)s.' % { - 'username': username, - 'remote_ip': utils.get_client_ip(self.request) - } - LOG.info(msg) + LOG.info('Login successful for user "%(username)s" using domain ' + '"%(domain)s", remote address %(remote_ip)s.', + {'username': username, 'domain': domain, + 'remote_ip': utils.get_client_ip(self.request)}) except exceptions.KeystoneAuthException as exc: - msg = 'Login failed for user "%(username)s", remote address '\ - '%(remote_ip)s.' % { - 'username': username, - 'remote_ip': utils.get_client_ip(self.request) - } - LOG.warning(msg) + LOG.info('Login failed for user "%(username)s" using domain ' + '"%(domain)s", remote address %(remote_ip)s.', + {'username': username, 'domain': domain, + 'remote_ip': utils.get_client_ip(self.request)}) raise forms.ValidationError(exc) if hasattr(self, 'check_for_test_cookie'): # Dropped in django 1.7 self.check_for_test_cookie()