stop spamming cadf event authn messages in log

Bug #1352115 reported by Steve Martinelli
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Fix Released
Low
Steve Martinelli

Bug Description

In keystone we currently log cadf events in the log, twice for every authN. Once for pending and once for success/failure.
This is unnecessary since the notification_driver option in keystone.conf can have multiple values, one of them being log.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to keystone (master)

Fix proposed to branch: master
Review: https://review.openstack.org/111634

Changed in keystone:
assignee: nobody → Steve Martinelli (stevemar)
status: New → In Progress
Dolph Mathews (dolph)
Changed in keystone:
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to keystone (master)

Reviewed: https://review.openstack.org/111634
Committed: https://git.openstack.org/cgit/openstack/keystone/commit/?id=13cb2eebaa01d0535ba42528bc71ce3a158dab93
Submitter: Jenkins
Branch: master

commit 13cb2eebaa01d0535ba42528bc71ce3a158dab93
Author: Steve Martinelli <email address hidden>
Date: Sun Aug 3 23:53:44 2014 -0400

    Remove debug CADF payload for every authN request

    Since we already provide a generic mechanism for logging notifications
    (setting the notification_driver value to 'log'), having an additional
    debug log just for this case seems unnecessary.

    Closes-Bug: #1352115
    Change-Id: I3bdf726ee2b32c434497b8c9372e2ab4cc8853cf

Changed in keystone:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in keystone:
milestone: none → juno-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in keystone:
milestone: juno-3 → 2014.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.