Hide Forgot
Description of problem: When a new user is created, the name, userid, email and group are NOT logged with correct info. They are just logged with "new_value" in audit.log which is not meaningful to use. Version-Release number of selected component (if applicable): cfme-5.6.0.13-1.el7cf.x86_64 How reproducible: 1. Select Settings -> Configuration 2. Expand the Access Control accordion 3. Select Users 4. Select Configuration -> Add a new user 5. Specify new user values and Add 6. Inspect the CloudForms audit log (/var/www/miq/.vmdb/log/audit.log) Steps to Reproduce: 1. NA 2. 3. Actual results: Not meaningful string "new_value" is specified for each new user attribute in the audit log Example: [----] I, [2016-11-14T08:04:03.179199 #23887:14b908c] INFO -- Success: MIQ(OpsRbac.rbac_edit_save_or_add) userid: [admin] - [New Admin] Record added ( name:[] to [new_value], userid:[] to [new_value], email:[] to [new_value], group:[] to [new_value]) Expected results: Values specified during user creation are shown in the audit log. Additional info:
Working on it, should have a fix over the weekend. Let me change status to On Dev. Greg Piatigorski
Code fix applied: https://github.com/ManageIQ/manageiq/pull/12786
New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/2948be72c1460738249b970d245d4e362760fcab commit 2948be72c1460738249b970d245d4e362760fcab Author: Greg Piatigorski <gpiatigo> AuthorDate: Mon Nov 21 16:38:53 2016 -0800 Commit: Greg Piatigorski <gpiatigo> CommitDate: Mon Nov 21 16:38:53 2016 -0800 Fix Audit Log to record settings/values when creating new user https://bugzilla.redhat.com/show_bug.cgi?id=1394593 app/controllers/ops_controller.rb | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)
Verified: 5.8.0.12-rc1.20170425180304_4f35996 audit.log now displays the data: [----] I, [2017-05-02T06:06:11.638948 #5793:17483fc] INFO -- Success: MIQ(OpsRbac.rbac_edit_save_or_add) userid: [admin] - [test] Record added ( name:[] to [test], group:[] to [18], userid:[] to [test] password:[*] to [*])