Bug 684259

Summary: incorrect group used for tps operators
Product: [Retired] Dogtag Certificate System Reporter: Ade Lee <alee>
Component: TPSAssignee: Ade Lee <alee>
Status: CLOSED CURRENTRELEASE QA Contact: Chandrasekar Kannan <ckannan>
Severity: high Docs Contact:
Priority: high    
Version: 9.0CC: aakkiang, alee, benl, dlackey, jmagne
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-06-04 20:35:59 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 445047    
Attachments:
Description Flags
patch to fix
cfu: review+
ui patch none

Description Ade Lee 2011-03-11 15:51:46 UTC
Description of problem:

When tps operator role was created, the ldap group tus officers was used to store users who were "operators".  This is incorrect because this group is supposed to be used for those users who are security officers and allowed to log into the security officers workstation.

A new group "tus operators" must be added and used instead.

 
Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 Ade Lee 2011-03-11 16:22:12 UTC
Created attachment 483767 [details]
patch to fix

Comment 2 Christina Fu 2011-03-11 16:39:06 UTC
Comment on attachment 483767 [details]
patch to fix

good catch.  Thanks.

Comment 3 Ade Lee 2011-03-11 20:25:08 UTC
8.1:

[vakwetu@goofy-vm4 tps]$ svn ci -m "Bugzilla BZ#684259:  incorrect group used for tps operators"
Sending        tps/scripts/addAgents.ldif
Sending        tps/src/include/tus/tus_db.h
Sending        tps/src/modules/tokendb/mod_tokendb.cpp
Sending        tps/src/tus/tus_db.c
Transmitting file data ....
Committed revision 1894.

tip:

[vakwetu@dhcp231-121 tps]$  svn ci -m "Bugzilla BZ#684259:  incorrect group used for tps operators"
Sending        tps/scripts/addAgents.ldif
Sending        tps/src/include/tus/tus_db.h
Sending        tps/src/modules/tokendb/mod_tokendb.cpp
Sending        tps/src/tus/tus_db.c
Transmitting file data ....
Committed revision 1895.

Comment 4 Ade Lee 2011-03-11 20:28:49 UTC
Note to docs:

This may have migration implications from 8.0-> 8.1.

Specifically, for existing instances, the admin will need to :

1. add a new group for the TUS Operators
2. add all the users that were operators to this new group - either through ldap or through the admin UI pages.

dn: cn=TUS Operators,ou=Groups,$TOKENDB_ROOT
objectClass: top
objectClass: groupOfNames
cn: TUS Operators
member: uid=admin,ou=People,$TOKENDB_ROOT
description: Operators for TUS

Comment 5 Ade Lee 2011-03-21 15:23:38 UTC
Created attachment 486627 [details]
ui patch

Comment 6 Ade Lee 2011-03-21 15:30:43 UTC
ui patch (tip):

[vakwetu@dhcp231-121 tps-ui]$ svn ci -m "Bugzilla BZ684259: wrong group used for tps operators"
Sending        tps-ui/shared/docroot/tokendb/editUser.template
Sending        tps-ui/shared/docroot/tokendb/newUser.template
Sending        tps-ui/shared/docroot/tokendb/userDelete.template
Transmitting file data .....
Committed revision 1902.

8.1:

[vakwetu@goofy-vm4 tps-ui]$ svn ci  -m "Bugzilla BZ684259: wrong group used for tps operators"
Sending        tps-ui/shared/docroot/tokendb/editUser.template
Sending        tps-ui/shared/docroot/tokendb/newUser.template
Sending        tps-ui/shared/docroot/tokendb/userDelete.template
Transmitting file data ...
Committed revision 15841.

Comment 9 Asha Akkiangady 2011-09-01 19:42:28 UTC
-Migration from CS 8.0 -> 8.1 created the TUS Operators group in the ldap db.
-After the migration adding existing users to the TUS operators group using TPS UI is successful.
-After the migration creating a new operator user in the TPS UI is successful.


Marking the bug verified.