Thanks Richard and Gerard for the quick changes, I will test the new patch soon.
Regards, Jiajia From: Richard Feezel [mailto:[email protected]] Sent: Thursday, March 31, 2016 7:50 PM To: Apache Directory Developers List Subject: Re: [jira] [Commented] (DIRKRB-542) Kerby Authorization I have completed a set of changes which addresses the induced problem in Asn1CollectionType. I have also made changes to the tests where the test was dependent on the incorrect behavior of this class. I have all tests passing now. Gerard should be able to post an additional patch with these changes today. On Thu, Mar 31, 2016 at 6:20 AM, Zheng, Kai <[email protected]<mailto:[email protected]>> wrote: Thanks Richard for digging into the deep! If the test failure is the only issue incurred by the desired change, we can fix the test anyway. But I’m afraid the impact may be more than that. I remembered I had considered to change the behavior, but due to some impacts I leave it as it was. When the patch can compile and have the basic issues resolved, I would look at the change and see how to fix the failure. Regards, Kai From: Richard Feezel [mailto:[email protected]<mailto:[email protected]>] Sent: Thursday, March 31, 2016 11:51 AM To: Apache Directory Developers List <[email protected]<mailto:[email protected]>> Subject: Re: [jira] [Commented] (DIRKRB-542) Kerby Authorization PersonnelRecordTest is indeed failing due to changes in Asn1CollectionType.java in the patch. Unfortunately we can't simply roll back the changes because the changes were made to address a problem with the decoding logic which resulted in adding default or empty values to the decoded object when fields were missing in the encoded data stream. This is incorrect behavior. Additional changes will be necessary to this class to correct the induced problem. On Wed, Mar 30, 2016 at 9:44 PM, Jiajia Li (JIRA) <[email protected]<mailto:[email protected]>> wrote: [ https://issues.apache.org/jira/browse/DIRKRB-542?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15219187#comment-15219187 ] Jiajia Li commented on DIRKRB-542: ---------------------------------- [~gg], thanks for your contribution, I think it will be new achievement for Kerby. Some issues: 1. After applying the patch to trunk, with some tests failure(PersonnelRecordTest, SignedDataTest...) 2. Compilation failure(ADAuthenticationIndicators.java) 3. The patch reformatted some original code, then some lines are longer than 80 characters(ps: Each line should not exceed 80) 4. A lot of " @SuppressWarnings("unchecked")" were added in the patch, I think some of them can be removed. > Kerby Authorization > ------------------- > > Key: DIRKRB-542 > URL: https://issues.apache.org/jira/browse/DIRKRB-542 > Project: Directory Kerberos > Issue Type: Sub-task > Reporter: Gerard Gagliano > Assignee: Gerard Gagliano > Attachments: ad.patch > > > Kerby lacks Authorization classes. Authorization types from RFC 1510, 4120, > 4537, 4556, 6711 and 7751 will greatly enhance the usability of Kerby. -- This message was sent by Atlassian JIRA (v6.3.4#6332) -- Richard M Feezel [email protected]<mailto:[email protected]> -- Richard M Feezel [email protected]<mailto:[email protected]>
