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

    https://github.com/apache/guacamole-client/pull/299#discussion_r194525708
  
    --- Diff: 
extensions/guacamole-auth-ldap/src/main/java/org/apache/guacamole/auth/ldap/connection/ConnectionService.java
 ---
    @@ -126,6 +127,10 @@
                 TokenFilter tokenFilter = new TokenFilter();
                 StandardTokens.addStandardTokens(tokenFilter, user);
     
    +            // Add custom attribute tokens
    +            Map<String, String> attrs = ( 
(org.apache.guacamole.auth.ldap.user.AuthenticatedUser) user).getAttributes();
    --- End diff --
    
    I'm not sure we want to have to do this type-casting, here, but see my 
comment below on the implementation within the LDAP-specific 
`AuthenticatedUser` class.


---

Reply via email to