[OPENDJ-962] Subject Attr To User Attr Cert Mapper has wrong default configuration Created: 03/Jun/13 Updated: 08/Nov/19 Resolved: 11/Jun/13 |
|
Status: | Done |
Project: | OpenDJ |
Component/s: | None |
Affects Version/s: | 2.6.0 |
Fix Version/s: | 2.6.0 |
Type: | Bug | Priority: | Minor |
Reporter: | manuelgaupp | Assignee: | Matthew Swift |
Resolution: | Fixed | Votes: | 0 |
Labels: | release-notes |
Issue Links: |
|
||||||||
Dev Assignee: | Matthew Swift |
Description |
The Subject Attribute To User Attribute Certificate Mapper is configured with the following default mappings:
I wasn't able to successfully map a certificate with the e:mail mapping and I doubt that it works because there is no attribute type e defined in the server's schema. This being said, I think that e refers to the emailAddress AttributeType from the PKCS#9 schema (IIRC it is displayed as E in many applications on Windows environments). There are 3 possible ways to fix this issue:
Solution 1 and 2 also require an update to the documentation to reflect the changes. Additionally, it would make sense that the isConfigurationAcceptable method also checks if certAttrName is valid (a valid OID or an attribute type which is defined in the server's schema). At the moment, only the validity of the userAttrName is being checked. |
Comments |
Comment by Matthew Swift [ 07/Jun/13 ] |
Hi Manuel, I see what you mean and I can see why it's easy to get confused. For example, when I look at the certificate details using Google Chrome I see "E: techies@forgerock.com", yet the attribute's name is definitely emailAddress, and this is what Java uses (see sun.security.x509.AVAKeyword). I'm happy with either approach (1) or (2), whichever you think is best. I don't think that we should make isConfigurationAcceptable stricter by checking that certAttrName is in the schema. The intention at the moment is that users should be able to configure the certificate mapper to cope with new types of certificate without being forced to update their schema. Matt |
Comment by manuelgaupp [ 11/Jun/13 ] |
Fixed in revision 8985 (source) and 8986 (admin guide). |
Comment by Ludovic Poitou [ 11/Jun/13 ] |
Thanks Manuel. |
Comment by manuelgaupp [ 11/Jun/13 ] |
I just changed the description of the emailAddress attributeType to make it more clear that it is used in X.509 certificates. |
Comment by Matthew Swift [ 07/Nov/19 ] |
Moved to closed state because the fixVersion has already been released. |