[keycloak-user] ldap novell eDirectory patch

Stian Thorgersen stian at redhat.com
Wed Apr 15 08:01:51 EDT 2015



----- Original Message -----
> From: "fiorenzo pizza" <fiorenzo.pizza at ict-group.it>
> To: "Marek Posolda" <mposolda at redhat.com>, keycloak-user at lists.jboss.org
> Sent: Wednesday, April 15, 2015 9:08:19 AM
> Subject: Re: [keycloak-user] ldap novell eDirectory patch
> 
> Hi Marek,
> 
> i will fix in the master and then i will send a PR.
> 
> I would like also add italian translation of messages
> (in forms-common-themes -> messages_it.properties).

We'd love to get a PR for Italian. However, for each language we add we need someone to volunteer to maintain it.

Would you be willing to track changes to messages_en.properties and apply to messages_it.properties?

> 
> 
> Fiorenzo
> 
> Il 15/04/15 08:28, Marek Posolda ha scritto:
> 
> 
> 
> Thanks for pointing this. We did not test with Novell eDirectory, so it's
> possible that it didn't work. If you have opportunity to try with latest
> Keycloak master, fix it in master and then send the PR, it will be very
> cool! Note that there was some refactoring in latest master and some code
> you pointed is not valid anymore (PartitionManagerRegistry was refactored
> and renamed to LDAPIdentityStoreRegistry).
> 
> 
> 
> 
> Otherwise I can try to apply it myself and ping you, so you can give it a try
> with novell edirectory (or wait for the next released keycloak version).
> 
> What do you think?
> Marek
> 
> On 15.4.2015 00:30, fiorenzo.pizza at ict-group.it wrote:
> 
> 
> Hi,
> 
> to support Novell eDirectory GUID, you can modify some lines of codes..
> 
> 1) [project keycloak-model-api] -> class: org.keycloak.models.LDAPConstants
> 
> - adding a new vendor constant ( public static final String
> VENDOR_NOVELL_EDIRECTORY="edirectory" ;)
> 
> 2) [project keycloak-picketlink-ldap] -> class:
> org.keycloak.picketlink.ldap.PartitionManagerRegistry
> 
> // RHDS is using "nsuniqueid" as unique identifier instead of "entryUUID"
> // Novell eDirectory use "guid"
> if (vendor != null && vendor.equals(LDAPConstants.VENDOR_RHDS)) {
> ldapStoreBuilder.uniqueIdentifierAttributeName("nsuniqueid");
> } else if (LDAPConstants.VENDOR_TIVOLI.equals(vendor)) {
> ldapStoreBuilder.uniqueIdentifierAttributeName("uniqueidentifier");
> } else if (LDAPConstants.VENDOR_NOVELL_EDIRECTORY.equals(vendor)) {
> ldapStoreBuilder.uniqueIdentifierAttributeName("guid");
> }
> 
> 3) [project keycloak-forms-common-themes] -> file: users.js
> 
> $scope.ldapVendors = [
> { "id": "ad", "name": "Active Directory" },
> { "id": "rhds", "name": "Red Hat Directory Server" },
> { "id": "tivoli", "name": "Tivoli" },
> { "id": "edirectory", "name": "eDirectory " },
> { "id": "other", "name": "Other" }
> ];
> 
> 
> What do you think about? I need more time to test my patch.. the keycloak
> project not compiles in my workspace...
> 
> Best Regards
> 
> Fiorenzo Pizza
> 
> 
> _______________________________________________
> keycloak-user mailing list keycloak-user at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/keycloak-user
> 
> 
> 
> _______________________________________________
> keycloak-user mailing list
> keycloak-user at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/keycloak-user


More information about the keycloak-user mailing list