openstack/horizon
Code Issues Proposed changes

Add user_domain_name to logs regarding authentication

Browse Source 
The user domain name is useful information when troubleshooting
authentication in OpenStack-installations with multiple domains.

Change-Id: I1cecd36bfafd7bdf9a9c68d2311fa5ff96272f36
This commit is contained in:
Trygve Vea 2017-11-23 20:37:00 +00:00
parent 846d269d90
commit b41402ef26
1 changed files with 8 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
openstack_auth/forms.py
View File

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