Github user mike-jumper commented on a diff in the pull request:

    https://github.com/apache/guacamole-client/pull/353#discussion_r245799935
  
    --- Diff: 
extensions/guacamole-auth-ldap/src/main/java/org/apache/guacamole/auth/ldap/connection/ConnectionService.java
 ---
    @@ -167,18 +201,34 @@
                                 // Parse name
                                 String name = parameter.substring(0, equals);
                                 String value = parameter.substring(equals+1);
    -
    -                            config.setParameter(name, value);
    +                            
    +                            // Pull out and set proxy parameters, if 
present
    +                            // Otherwise set the parameter.
    +                            switch(name) {
    +                                case PROXY_HOST_PARAMETER:
    --- End diff --
    
    Depending on OID assignment, it looks like we should be fine with the above 
system. There is no standard limit imposed on OID length, and longer OIDs are 
already actively used.


---

Reply via email to