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

    https://github.com/apache/guacamole-client/pull/204#discussion_r161387157
  
    --- Diff: 
extensions/guacamole-auth-ldap/src/main/java/org/apache/guacamole/auth/ldap/AuthenticationProviderService.java
 ---
    @@ -124,10 +126,15 @@ private String getUserBindDN(String username)
                     }
     
                     // Return the single possible DN
    -                return userDNs.get(0);
    +                return new Dn(userDNs.get(0));
    --- End diff --
    
    Good idea - let me see if there's any reason not to do this.


---

Reply via email to