openstack/keystone
Code Issues Proposed changes

Remove unused exceptions from CADF notifications

Browse Source 
This change removes exceptions that are never thrown during
authentication due to them being related to password validation
and are caught/handled elsewhere from this change:

https://review.openstack.org/#/c/396752/

Change-Id: Iab49986ec41028da5c0d45f83c0037c07434c744
This commit is contained in:
Gage Hugo 2016-12-21 18:17:33 -06:00 committed by Steve Martinelli
parent a5efb16256
commit ddff3bdf0a
1 changed files with 1 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
keystone/notifications.py
View File

@ -492,10 +492,7 @@ class CadfNotificationWrapper(object):
try: try:
result = f(wrapped_self, request, user_id, *args, **kwargs) result = f(wrapped_self, request, user_id, *args, **kwargs)
except (exception.AccountLocked, except (exception.AccountLocked,
exception.PasswordExpired, exception.PasswordExpired) as ex:
exception.PasswordRequirementsValidationError,
exception.PasswordHistoryValidationError,
exception.PasswordAgeValidationError) as ex:
# Send a CADF event with a reason for PCI-DSS related # Send a CADF event with a reason for PCI-DSS related
# authentication failures # authentication failures
audit_reason = reason.Reason(str(ex), str(ex.code)) audit_reason = reason.Reason(str(ex), str(ex.code))