On 04/29/2012 09:49 AM, Oved Ourfalli wrote: > Hey, > > In case the namingContexts field contains more than one naming context, we > have no way of knowing which one is the required one. > We take the first one. In active directory, for example (and as far as I > understand in the newest free-ipa) you have a defaultNamingContext field, and > if so we take it, and it contains only one naming context. > > As you see in the code below, we have a "special" treatment for RHDS, as it > always contains "o=netscaperoot" in the namingContexts field (not necessary > at the beginning of the list), so we filter it out. > > It was enough for now, but I agree that if adding new LDAP providers, which > usually have a more complex and unpredictable data in the namingContext > field, then we would probably need to ask the user to specify the naming > context for us, and if he didn't specify it then assume it is identical to > the domain name. > > Oved
Another idea - Now that we provide the provider type in manage domains, maybe we should add a logic for getting naming contexts based on the provider type. Thoughts on this? > > ----- Original Message ----- >> From: "Sharad Mishra" <[email protected]> >> To: [email protected] >> Sent: Saturday, April 28, 2012 2:51:25 AM >> Subject: [Engine-devel] namingcontexts getting ignored. >> >> >> Hi, >> >> I enter "getDefaultNamingContextFromNamingContexts()" with >> namingContexts set to - >> >> namingcontexts: CN=SCHEMA, CN=LOCALHOST, CN=IBMPOLICIES, O=IBM.COM, >> O=DELETED.IBM.COM >> >> But this method only takes the first case "CN=SCHEMA" and ignores >> the rest. Actually I am interested in "O=IBM.COM". >> Is this working as designed or needs improvement? >> >> -Sharad Mishra >> IBM >> >> public class RootDSEData { >> private String domainDN = null; >> private LdapProviderType ldapProviderType = null; >> private final static String RHDS_NAMING_CONTEXT = >> "o=netscaperoot"; >> >> public static String >> getDefaultNamingContextFromNamingContexts(Attribute namingContexts) { >> for (int index = 0; index < namingContexts.size(); ++index) >> { >> String namingContext; >> try { >> namingContext = (String) namingContexts.get(index); >> } catch (NamingException e) { >> return null; >> } >> if >> (!RHDS_NAMING_CONTEXT.equalsIgnoreCase(namingContext)) >> { >> return namingContext; >> } >> } >> return null; >> } >> >> _______________________________________________ >> Engine-devel mailing list >> [email protected] >> http://lists.ovirt.org/mailman/listinfo/engine-devel >> > _______________________________________________ > Engine-devel mailing list > [email protected] > http://lists.ovirt.org/mailman/listinfo/engine-devel _______________________________________________ Engine-devel mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-devel
