Uploaded image for project: 'LDAP Connector'
  1. LDAP Connector
  2. MGNLLDAP-101

LDAP tester is broken after introducing the cache into LDAPUserManager

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Neutral Neutral
    • 1.8
    • None
    • None
    • Kromeriz 80
    • 5

      Fix:

      • Disable cache for ldap tester
      • info.magnolia.jaas.sp.ldap.LDAPUserManager#loadProperties should use methods info.magnolia.jaas.sp.ldap.LDAPUserManager#getItemFromCache and info.magnolia.jaas.sp.ldap.LDAPUserManager#addItemIntoCache which checks if cache is not disabled and are failsafe if ldap-user-manager-cache can't be initialized
      • Add missing dependency to cache-core into ldap-tester pom
      • Put back info.magnolia.ldap.tool.ShadeSmoker which is used to trick the shade-plugin (to not remove ldap module classes when they are not referenced directly from code and minimizeJar property is set to true)

      Improvements:

      • use user.dir instead of hardcoded /tmp folder
      • set realmName for userManager
      • add unit test for Tester

        Acceptance criteria

              mdivilek Milan Divilek
              had Jan Haderka
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated:
                Resolved:

                  Bug DoR
                  Task DoD