[GitHub] guacamole-client pull request #345: GUACAMOLE-234: Migrate to Apache Directo...

[necouchman]

Github user necouchman commented on a diff in the pull request:

    https://github.com/apache/guacamole-client/pull/345#discussion_r241916443
  
    --- Diff: 
extensions/guacamole-auth-ldap/src/main/java/org/apache/guacamole/auth/ldap/AuthenticationProviderService.java
 ---
    @@ -240,17 +244,24 @@ public LDAPAuthenticatedUser 
authenticateUser(Credentials credentials)
     
             try {
     
    +            LdapConnectionConfig ldapConnectionConfig =
    +                    ((LdapNetworkConnection) ldapConnection).getConfig();
    +            Dn authDn = new Dn(ldapConnectionConfig.getName());
    --- End diff --
    
    I'll take a look and try to figure something out.


---