Return-Path: X-Original-To: apmail-directory-commits-archive@www.apache.org Delivered-To: apmail-directory-commits-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 74BFA17236 for ; Tue, 28 Oct 2014 11:20:49 +0000 (UTC) Received: (qmail 67183 invoked by uid 500); 28 Oct 2014 11:20:49 -0000 Delivered-To: apmail-directory-commits-archive@directory.apache.org Received: (qmail 67146 invoked by uid 500); 28 Oct 2014 11:20:49 -0000 Mailing-List: contact commits-help@directory.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@directory.apache.org Delivered-To: mailing list commits@directory.apache.org Received: (qmail 67137 invoked by uid 99); 28 Oct 2014 11:20:49 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 28 Oct 2014 11:20:49 +0000 X-ASF-Spam-Status: No, hits=-2000.0 required=5.0 tests=ALL_TRUSTED X-Spam-Check-By: apache.org Received: from [140.211.11.4] (HELO eris.apache.org) (140.211.11.4) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 28 Oct 2014 11:20:44 +0000 Received: from eris.apache.org (localhost [127.0.0.1]) by eris.apache.org (Postfix) with ESMTP id 28C422388993; Tue, 28 Oct 2014 11:18:54 +0000 (UTC) Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: svn commit: r1634848 - /directory/site/trunk/content/fortress/issues.mdtext Date: Tue, 28 Oct 2014 11:18:54 -0000 To: commits@directory.apache.org From: elecharny@apache.org X-Mailer: svnmailer-1.0.9 Message-Id: <20141028111854.28C422388993@eris.apache.org> X-Virus-Checked: Checked by ClamAV on apache.org Author: elecharny Date: Tue Oct 28 11:18:53 2014 New Revision: 1634848 URL: http://svn.apache.org/r1634848 Log: Added fortress issues 1* Modified: directory/site/trunk/content/fortress/issues.mdtext Modified: directory/site/trunk/content/fortress/issues.mdtext URL: http://svn.apache.org/viewvc/directory/site/trunk/content/fortress/issues.mdtext?rev=1634848&r1=1634847&r2=1634848&view=diff ============================================================================== --- directory/site/trunk/content/fortress/issues.mdtext (original) +++ directory/site/trunk/content/fortress/issues.mdtext Tue Oct 28 11:18:53 2014 @@ -668,6 +668,1191 @@ hr.fullcontent
+ + + + + + + + + + + + + + + + + + + + + + + + + + +
+

+ [FC-10] UserDAO throws npe when password is null + Created: 18/May/13 Updated: 18/May/13 Due: 20/May/13 Resolved: 18/May/13 +

+
Status:Resolved
Project:Fortress Core
Component/s:None
Affects Version/s:None
Fix Version/s:1.0-RC26
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
Type:BugPriority:Major
Reporter:Shawn McKinney [Administrator]Assignee:Shawn McKinney [Administrator]
Resolution:FixedVotes:0
Labels:None
Remaining Estimate:0 minutes
Time Spent:1 hour
Original Estimate:Not Specified
Environment: +

all

+
+ + + + + +
Attachments: + PNG File + UserDAO-PW-NPE-20130517.png     +
+ + + + + +
 Description  
+ + + + +
+

The UserDAO createUser and updateUpdate user methods throw npe when user entity contains a null value in password field.

+
+
+ + + + + +
 Comments  +  
+ + + + + + + +
Comment by Shawn McKinney [Administrator][18/May/13]
+

fix npe for null password

+
+
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + +
+

+ [FC-11] Builder change to pull source from maven + Created: 19/May/13 Updated: 28/Feb/14 Resolved: 28/Feb/14 +

+
Status:Closed
Project:Fortress Core
Component/s:None
Affects Version/s:None
Fix Version/s:None
+ + + + >Fixed + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
Type:ImprovementPriority:Major
Reporter:Shawn McKinney [Administrator]Assignee:Shawn McKinney [Administrator]
Resolution:Won't FixVotes:0
Labels:None
Remaining Estimate:1 day
Time Spent:Not Specified
Original Estimate:1 day
Environment: +

all

+
+ +
+ + + + + +
 Description  
+ + + + +
+

Change quickstart package's 'builder' target to pull fortress source from maven. Do not include source in quickstart archive.

+
+
+
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + +
+

+ [FC-12] isTemporalSet does not detect changes to entity constraint values + Created: 26/May/13 Updated: 26/May/13 Due: 31/May/13 Resolved: 26/May/13 +

+
Status:Resolved
Project:Fortress Core
Component/s:None
Affects Version/s:None
Fix Version/s:1.0-RC26
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
Type:BugPriority:Major
Reporter:Shawn McKinney [Administrator]Assignee:Shawn McKinney [Administrator]
Resolution:FixedVotes:0
Labels:None
Remaining Estimate:Not Specified
Time Spent:Not Specified
Original Estimate:Not Specified
Environment: +

all

+
+ +
+ + + + + +
 Description  
+ + + + +
+

fix this method to detect when any of the constraint attributes changes. Currently it will only trigger if all of the attributes are not null.

+

public boolean isTemporalSet()

+ { + return (beginTime != null && endTime != null && beginDate != null && endDate != null && beginLockDate != null && endLockDate != null && dayMask != null); + } +
+
+ + + + + +
 Comments  +  
+ + + + + + + +
Comment by Shawn McKinney [Administrator][26/May/13]
+

Changed the && to ||

+
+
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + +
+

+ [FC-13] ReviewMgr.findUsers does not pull back pw policy subentry + Created: 28/May/13 Updated: 28/May/13 Due: 29/May/13 Resolved: 28/May/13 +

+
Status:Resolved
Project:Fortress Core
Component/s:None
Affects Version/s:None
Fix Version/s:1.0-RC26
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
Type:BugPriority:Major
Reporter:Shawn McKinney [Administrator]Assignee:Shawn McKinney [Administrator]
Resolution:FixedVotes:0
Labels:None
Remaining Estimate:1 hour
Time Spent:Not Specified
Original Estimate:1 hour
Environment: +

all

+
+ +
+ + + + + +
 Description  
+ + + + +
+

Add OPENLDAP_POLICY_SUBENTRY to the result set for user search

+
+
+ + + + + +
 Comments  +  
+ + + + + + + +
Comment by Shawn McKinney [Administrator][28/May/13]
+

added attribute to search

+
+
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + +
+

+ [FC-14] Add bldg, dept and room attrs to user entity + Created: 28/May/13 Updated: 03/Jun/13 Due: 31/May/13 Resolved: 03/Jun/13 +

+
Status:Resolved
Project:Fortress Core
Component/s:None
Affects Version/s:None
Fix Version/s:1.0-RC26
+ + + + >Fixed + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
Type:ImprovementPriority:Major
Reporter:Shawn McKinney [Administrator]Assignee:Shawn McKinney [Administrator]
Resolution:FixedVotes:0
Labels:None
Remaining Estimate:4 hours
Time Spent:Not Specified
Original Estimate:4 hours
Environment: +

all

+
+ +
+ + + + + +
 Description  
+ + + + +
+

Add inetorgperson attributes bldg, dept and room to user

+
+
+ + + + + +
 Comments  +  
+ + + + + + + +
Comment by Shawn McKinney [Administrator][03/Jun/13]
+

done

+
+
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + +
+

+ [FC-15] Fortress UserDAO.createUser adds openldap policy flag when server type = apache ds + Created: 29/May/13 Updated: 29/May/13 Due: 30/May/13 Resolved: 29/May/13 +

+
Status:Resolved
Project:Fortress Core
Component/s:None
Affects Version/s:None
Fix Version/s:1.0-RC26
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
Type:BugPriority:Major
Reporter:Shawn McKinney [Administrator]Assignee:Shawn McKinney [Administrator]
Resolution:FixedVotes:0
Labels:None
Remaining Estimate:Not Specified
Time Spent:Not Specified
Original Estimate:Not Specified
Environment: +

all

+
+ +
+ + + + + +
 Description  
+ + + + +
+

final User create(User entity)
+ {
+ add flag here:
+ if (VUtil.isNotNullOrEmpty(entity.getPwPolicy()))

+ { + String dn = GlobalIds.POLICY_NODE_TYPE + "=" + entity.getPwPolicy() + "," + getRootDn(entity.getContextId(), GlobalIds.PPOLICY_ROOT); + attrs.add(createAttribute(OPENLDAP_POLICY_SUBENTRY, dn)); + } +
+
+ + + + + +
 Comments  +  
+ + + + + + + +
Comment by Shawn McKinney [Administrator][29/May/13]
+

added hooks to prevent pulling OL attributes when server.type = apacheds

+
+
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + +
+

+ [FC-16] MDB Error during deletion + Created: 29/May/13 Updated: 03/Jun/13 Due: 07/Jun/13 +

+
Status:Open
Project:Fortress Core
Component/s:None
Affects Version/s:None
Fix Version/s:None
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
Type:BugPriority:Major
Reporter:Shawn McKinney [Administrator]Assignee:Shawn McKinney [Administrator]
Resolution:UnresolvedVotes:0
Labels:None
Remaining Estimate:Not Specified
Time Spent:Not Specified
Original Estimate:Not Specified
Environment: +

ubuntu32

+
+ + + + + +
Attachments: + Text File + konsole-output-mdb-delete-error-20130529.txt     +
+ + + + + +
 Description  
+ + + + +
+

During deletion to fortress permission object:
+ ftObjNm=TOB4_8,ou=Permissions,ou=RBAC,dc=jts,dc=us

+

PermDAO.delete

+

This error in debug level slapd.logs, excerpt below, full log attached:

+

51a66b8d => index_entry_del( 911, "ftObjNm=TOB4_8,ou=Permissions,ou=RBAC,dc=jts,dc=us" )

+

51a66b8d mdb_idl_delete_keys: 38f 00000000
+ 51a66b8d mdb_idl_delete_keys: 38f 0096defd
+ 51a66b8d mdb_idl_delete_keys: 38f 3d1dea68
+ 51a66b8d mdb_idl_delete_keys: 38f 76324484
+ 51a66b8d mdb_idl_delete_keys: 38f 29c95ac5
+ 51a66b8d mdb_idl_delete_keys: 38f 0e1b3d46
+ 51a66b8d mdb_idl_delete_keys: 38f 1ccd25c5
+ 51a66b8d <= index_entry_del( 911, "ftObjNm=TOB4_8,ou=Permissions,ou=RBAC,dc=jts,dc=us" ) success
+ 51a66b8d <=- mdb_delete: id2entry failed: MDB_PAGE_FULL: Internal error - page has no more space (-30786)
+ 51a66b8d send_ldap_result: conn=1000 op=3820 p=3
+ 51a66b8d send_ldap_result: err=80 matched="" text="entry delete failed"
+ 51a66b8d slap_queue_csn: queing 0x3bb8e308 20130529205645.303808Z#000000#000#000000
+ 51a66b8d ==> mdb_add: reqStart=20130529205645.000000Z,cn=log
+ 51a66b8d oc_check_required entry (reqStart=20130529205645.000000Z,cn=log), objectClass "auditDelete"
+ 51a66b8d oc_check_allowed type "objectClass"
+ 51a66b8d oc_check_allowed type "structuralObjectClass"
+ 51a66b8d oc_check_allowed type "reqStart"
+ 51a66b8d oc_check_allowed type "reqEnd"
+ 51a66b8d oc_check_allowed type "reqType"
+ 51a66b8d oc_check_allowed type "reqSession"
+ 51a66b8d oc_check_allowed type "reqAuthzID"
+ 51a66b8d oc_check_allowed type "reqDN"
+ 51a66b8d oc_check_allowed type "reqMessage"
+ 51a66b8d oc_check_allowed type "reqResult"
+ 51a66b8d oc_check_allowed type "reqEntryUUID"
+ 51a66b8d mdb_dn2entry("reqStart=20130529205645Z,cn=log")
+ 51a66b8d => mdb_dn2id("reqStart=20130529205645Z,cn=log")
+ 51a66b8d <= mdb_dn2id: get failed: MDB_NOTFOUND: No matching key/data pair found (-30798)
+ 51a66b8d => mdb_entry_decode:
+ 51a66b8d <= mdb_entry_decode
+ 51a66b8d => access_allowed: add access to "cn=log" "children" requested
+ 51a66b8d <= root access granted
+ 51a66b8d => access_allowed: add access granted by manage(=mwrscxd)
+ 51a66b8d => access_allowed: add access to "reqStart=20130529205645.000000Z,cn=log" "entry" requested
+ 51a66b8d <= root access granted
+ 51a66b8d => access_allowed: add access granted by manage(=mwrscxd)
+ 51a66b8d => mdb_dn2id_add 0x4eba: "reqStart=20130529205645Z,cn=log"
+ 51a66b8d <= mdb_dn2id_add 0x4eba: 0
+ 51a66b8d => index_entry_add( 20154, "reqStart=20130529205645.000000Z,cn=log" )
+ 51a66b8d mdb_idl_insert_keys: 4eba b5c866dc
+ 51a66b8d mdb_idl_insert_keys: 4eba
+ 51a66b8d mdb_idl_insert_keys: 4eba 86ee7ec7
+ 51a66b8d mdb_idl_insert_keys: 4eba 7990d2ba
+ 51a66b8d <= index_entry_add( 20154, "reqStart=20130529205645.000000Z,cn=log" ) success
+ 51a66b8d => mdb_entry_encode(0x00004eba): reqStart=20130529205645.000000Z,cn=log
+ 51a66b8d <= mdb_entry_encode(0x00004eba): reqStart=20130529205645.000000Z,cn=log
+ 51a66b8d mdb_add: added id=00004eba dn="reqStart=20130529205645.000000Z,cn=log"
+ 51a66b8d send_ldap_result: conn=1000 op=3820 p=3
+ 51a66b8d send_ldap_result: err=0 matched="" text=""
+ 51a66b8d slap_graduate_commit_csn: removing 0x89e6d50 20130529205645.303808Z#000000#000#000000
+ 51a66b8d send_ldap_response: msgid=3821 tag=107 err=80
+ ber_flush2: 34 bytes to sd 14
+ 0000: 30 20 02 02 0e ed 6b 1a 0a 01 50 04 00 04 13 65 0 ....k...P....e
+ 0010: 6e 74 72 79 20 64 65 6c 65 74 65 20 66 61 69 6c ntry delete fail
+ 0020: 65 64 ed
+ ldap_write: want=34, written=34
+ 0000: 30 20 02 02 0e ed 6b 1a 0a 01 50 04 00 04 13 65 0 ....k...P....e
+ 0010: 6e 74 72 79 20 64 65 6c 65 74 65 20 66 61 69 6c ntry delete fail
+ 0020: 65 64 ed
+ 51a66b8d conn=1000 op=3820 RESULT tag=107 err=80 text=entry delete failed
+ 51a66b8d slap_graduate_commit_csn: removing 0x8a07120 20130529205645.303808Z#000000#000#000000
+ 51a66c46 daemon: epoll: listen=7 active_threads=0 tvp=zero

+
+
+
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + +
+

+ [FC-17] Delete descendant role error + Created: 29/May/13 Updated: 03/Jun/13 Due: 31/May/13 +

+
Status:Open
Project:Fortress Core
Component/s:None
Affects Version/s:None
Fix Version/s:None
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
Type:BugPriority:Major
Reporter:Shawn McKinney [Administrator]Assignee:Shawn McKinney [Administrator]
Resolution:UnresolvedVotes:0
Labels:None
Remaining Estimate:Not Specified
Time Spent:Not Specified
Original Estimate:Not Specified
Environment: +

ubuntu 32 - symas-openldap-silver.32_2.4.35-2_i386.deb

+
+ +
+ + + + + +
 Description  
+ + + + +
+

junit 2013-05-29 17:51:18,449 (INFO ) DEL-RLS-TR6-DESC
+ junit 2013-05-29 17:51:18,480 (ERROR) us.jts.fortress.rbac.AdminMgrImplTest.delRoleDescendant caught SecurityException rc=5004, msg=us.jts.fortress.rbac.RoleDAO.remove role name=oamT6C4B2A1 LDAPException=32 msg=no such object
+ junit us.jts.fortress.RemoveException: us.jts.fortress.rbac.RoleDAO.remove role name=oamT6C4B2A1 LDAPException=32 msg=no such object
+ junit at us.jts.fortress.rbac.RoleDAO.remove(RoleDAO.java:316)
+ junit at us.jts.fortress.rbac.RoleP.delete(RoleP.java:246)
+ junit at us.jts.fortress.rbac.AdminMgrImpl.deleteRole(AdminMgrImpl.java:434)
+ junit at us.jts.fortress.rbac.AdminMgrImplTest.delRoleDescendant(AdminMgrImplTest.java:670)
+ junit at us.jts.fortress.rbac.AdminMgrImplTest.testDelRoleDescendant(AdminMgrImplTest.java:603)
+ junit at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
+ junit at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
+ junit at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
+ junit at java.lang.reflect.Method.invoke(Method.java:601)
+ junit at junit.framework.TestCase.runTest(TestCase.java:168)
+ junit at junit.framework.TestCase.runBare(TestCase.java:134)
+ junit at junit.framework.TestResult$1.protect(TestResult.java:110)
+ junit at junit.framework.TestResult.runProtected(TestResult.java:128)
+ junit at junit.framework.TestResult.run(TestResult.java:113)
+ junit at junit.framework.TestCase.run(TestCase.java:124)
+ junit at junit.framework.TestSuite.runTest(TestSuite.java:243)
+ junit at junit.framework.TestSuite.run(TestSuite.java:238)
+ junit at org.apache.tools.ant.taskdefs.optional.junit.JUnitTestRunner.run(JUnitTestRunner.java:518)
+ junit at org.apache.tools.ant.taskdefs.optional.junit.JUnitTestRunner.launch(JUnitTestRunner.java:1052)
+ junit at org.apache.tools.ant.taskdefs.optional.junit.JUnitTestRunner.main(JUnitTestRunner.java:906)
+ junit Caused by: LDAPException(resultCode=32 (no such object), errorMessage='no such object', matchedDN='ou=Roles,ou=RBAC,dc=jts,dc=us')
+ junit at com.unboundid.ldap.sdk.migrate.ldapjdk.LDAPConnection.modify(LDAPConnection.java:1137)
+ junit at com.unboundid.ldap.sdk.migrate.ldapjdk.LDAPConnection.modify(LDAPConnection.java:1154)
+ junit at us.jts.fortress.ldap.DataProvider.modify(DataProvider.java:188)
+ junit at us.jts.fortress.ldap.DataProvider.delete(DataProvider.java:235)
+ junit at us.jts.fortress.rbac.RoleDAO.remove(RoleDAO.java:311)
+ junit ... 19 more
+ junit 2013-05-29 17:51:18,483 (INFO ) DEL-RLS-TR6-ASC
+ junit 2013-05-29 17:51:18,487 (ERROR) us.jts.fortress.rbac.AdminMgrImplTest.delRoleAscendant caught SecurityException rc=5060, msg=us.jts.fortress.rbac.HierUtil.validateRelationship child oamT7D2C1B1A1 does not have parent oamT7C2B1A1
+ junit us.jts.fortress.ValidationException: us.jts.fortress.rbac.HierUtil.validateRelationship child oamT7D2C1B1A1 does not have parent oamT7C2B1A1
+ junit at us.jts.fortress.rbac.HierUtil.validateRelationship(HierUtil.java:127)
+ junit at us.jts.fortress.rbac.RoleUtil.validateRelationship(RoleUtil.java:245)
+ junit at us.jts.fortress.rbac.AdminMgrImpl.deleteInheritance(AdminMgrImpl.java:1046)
+ junit at us.jts.fortress.rbac.AdminMgrImplTest.delRoleAscendant(AdminMgrImplTest.java:764)
+ junit at us.jts.fortress.rbac.AdminMgrImplTest.testDelRoleAscendant(AdminMgrImplTest.java:741)
+ junit at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
+ junit at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
+ junit at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
+ junit at java.lang.reflect.Method.invoke(Method.java:601)
+ junit at junit.framework.TestCase.runTest(TestCase.java:168)
+ junit at junit.framework.TestCase.runBare(TestCase.java:134)
+ junit at junit.framework.TestResult$1.protect(TestResult.java:110)
+ junit at junit.framework.TestResult.runProtected(TestResult.java:128)
+ junit at junit.framework.TestResult.run(TestResult.java:113)
+ junit at junit.framework.TestCase.run(TestCase.java:124)
+ junit at junit.framework.TestSuite.runTest(TestSuite.java:243)
+ junit at junit.framework.TestSuite.run(TestSuite.java:238)
+ junit at org.apache.tools.ant.taskdefs.optional.junit.JUnitTestRunner.run(JUnitTestRunner.java:518)
+ junit at org.apache.tools.ant.taskdefs.optional.junit.JUnitTestRunner.launch(JUnitTestRunner.java:1052)
+ junit at org.apache.tools.ant.taskdefs.optional.junit.JUnitTestRunner.main(JUnitTestRunner.java:906)
+ junit 2013-05-29 17:51:18,487 (INFO ) DEL-RLS TR1
+ junit 2013-05-29 17:51:18,502 (INFO ) DEL-RLS TR2
+ junit 2013-05-29 17:51:18,516 (INFO ) DEL-RLS TR3
+ junit 2013-05-29 17:51:18,771 (INFO ) DEL-RLS TR4
+ junit 2013-05-29 17:51:18,799 (INFO ) DEL-RLS ROLES_TR5_HIER
+ junit 2013-05-29 17:51:18,818 (INFO ) DEL-RLS ROLES_TR5B
+ junit 2013-05-29 17:51:18,852 (INFO ) DEL-RLS ROLES_TR8_SSD
+ junit 2013-05-29 17:51:18,919 (INFO ) DEL-RLS ROLES_TR9_SSD
+ ^Csmckinn@smckinnlt-lr02:~/GIT/fortressDev/openldap-fortress-core$ sudo ./build.sh init-slapd
+ Buildfile: /home/smckinn/GIT/fortressDev/openldap-fortress-core/build.xml

+
+
+
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + +
+

+ [FC-18] DAO Updates enhancement + Created: 30/May/13 Updated: 17/Jun/13 Due: 30/Jun/13 +

+
Status:Open
Project:Fortress Core
Component/s:None
Affects Version/s:None
Fix Version/s:None
+ + + + >Fixed + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
Type:ImprovementPriority:Major
Reporter:Shawn McKinney [Administrator]Assignee:Shawn McKinney [Administrator]
Resolution:UnresolvedVotes:0
Labels:None
Remaining Estimate:Not Specified
Time Spent:Not Specified
Original Estimate:Not Specified
Environment: +

all

+
+ +
+ + + + + +
 Description  
+ + + + +
+

09:15:48 AM emmanuel lecharny: I may have a few q regarding the update methods
+ 09:16:08 AM emmanuel lecharny: it seems that you remove an Attribute before adding some new values
+ 09:18:43 AM emmanuel lecharny: something like :
+ 09:18:44 AM emmanuel lecharny: LDAPModification(type=replace, attr=ftRoles, values={}),
+ LDAPModification(type=add, attr=ftRoles, values=

+ {'oamT10SSDR1'})]
+
+ 09:19:16 AM emmanuel lecharny: which results in two modification bing done on the entry, instead of one
+ 09:19:22 AM emmanuel lecharny: not a big deal though
+ 09:19:47 AM emmanuel lecharny: I assume that all the ftRoles get removed first, then you inject one new one
+ 09:20:06 AM * emmanuel lecharny out for 5 mins
+ 09:33:24 AM smckinney: so you are saying two roundtrips for those operations?
+ 09:34:22 AM smckinney: the intent is yes to replace the old with the new
+ 09:40:49 AM emmanuel lecharny: smckinney: no, this will be done in one single rountrip. This is why t's not a big deal
+ 09:41:00 AM smckinney: yes that was my assumption as well
+ 09:41:16 AM smckinney: but having your eyes in there will be good
+ 09:42:23 AM smckinney: how would you have done this?
+ 09:44:55 AM emmanuel lecharny: a replace with thevalues will blank the existing data, and inject the added values
+ 09:45:10 AM emmanuel lecharny: so no eed to do LDAPModification(type=replace, attr=ftRoles, values={}),
+
+ 09:45:39 AM emmanuel lecharny: doing LDAPModification(type=replace, attr=ftRoles, values={'oamT10SSDR1'} +

)] should be enough

+

09:46:27 AM smckinney: yes agreed. wondering why I did not do that to begin with...
+ 09:47:28 AM smckinney: will create an issue to take another look at it.

+
+
+
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + +
+

+ [FC-19] Add toString() methods for the base objects (Role, Permissions...) + Created: 30/May/13 Updated: 03/Jun/13 +

+
Status:Open
Project:Fortress Core
Component/s:None
Affects Version/s:None
Fix Version/s:1.0
+ + + + >Fixed + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
Type:ImprovementPriority:Major
Reporter:Emmanuel LecharnyAssignee:Shawn McKinney [Administrator]
Resolution:UnresolvedVotes:0
Labels:None
Remaining Estimate:Not Specified
Time Spent:Not Specified
Original Estimate:Not Specified
+ +
+ + + + + +
 Description  
+ + + + +
+

It would be very comfy to have toString() methods added to the base classes, especially for debugging and logging purpose.

+
+
+
+ +