There was a general problem with the ASN.1 decoding code prior to our patch. Objects were being created even though the corresponding (optional) tag never appeared in the ASN.1 byte stream being decoded. In the case of a SEQUENCE-OF an empty container was being created. This did not correctly reflect what was present (or not) in the byte stream being decoded. We fixed this so that objects are only created when the corresponding tag is actually present in the byte stream. This, however, broke some of the handling code. In those cases where an optional collection was being processed, previously the collection object would be present but have no members. NOW, the collection object itself is NULL if the corresponding tag wasn't present in the byte stream. This means that IF tests for NULL needed to be added in various places in the code.
Authorization Data is generally site-specific and therefore not inherently a part of Kerby. But the Kerberos protocol supports the inclusion, transport, and forwarding of authorization data in Kerberos tickets. For example, a Microsoft KDC will add authorization data to a ticket which lists the various groups a user is a member of (along with other details as well). Ideally Kerby would include a plug-able framework in which site implementors would be able to include modules which would inject, process, and forward authorization data contained in the tickets issued and received by Kerby. For now we have simply extended the Backend interface to allow a backend implementation to do the authorization data handling. This is not necessarily the ideal place for a "generalized" implementation of authorization data handling. However, there is likely to be a tight coupling between authorization data handling and the backend implementation, so we put this one method in to allow the backend to handle the authorization data. On Wed, May 25, 2016 at 7:41 AM, Gerard Gagliano (JIRA) <[email protected]> wrote: > > [ > https://issues.apache.org/jira/browse/DIRKRB-542?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15299923#comment-15299923 > ] > > Gerard Gagliano commented on DIRKRB-542: > ---------------------------------------- > > > > This is related to the Asn1 modifications. Items were filled in where > there were none, which is invalid and makes it impossible to re-encode. > > > It is an abstract class. You would override it and call the method. If > you weren’t wanting Authorization Data, the null is all the rest of the > code needs. > > > > > > 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: ADAll.patch, ad.patch, ad2.patch, ad3.patch, > adtest.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]
