directory-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Zheng, Kai" <kai.zh...@intel.com>
Subject RE: [jira] [Commented] (DIRKRB-542) Kerby Authorization
Date Thu, 31 Mar 2016 10:20:18 GMT
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:rfeezel@gmail.com]
Sent: Thursday, March 31, 2016 11:51 AM
To: Apache Directory Developers List <dev@directory.apache.org>
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) <jira@apache.org<mailto:jira@apache.org>>
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
rfeezel@gmail.com<mailto:rfeezel@gmail.com>
Mime
View raw message