necouchman commented on a change in pull request #353:
URL: https://github.com/apache/guacamole-client/pull/353#discussion_r642148001
##########
File path:
extensions/guacamole-auth-ldap/src/main/java/org/apache/guacamole/auth/ldap/connection/ConnectionService.java
##########
@@ -59,6 +62,42 @@
* Logger for this class.
*/
private static final Logger logger =
LoggerFactory.getLogger(ConnectionService.class);
+
+ /**
+ * The name of the LDAP attribute that stores connection configuration
+ * parameters for Guacamole.
+ */
+ public static final String LDAP_ATTRIBUTE_PARAMETER =
"guacConfigParameter";
+
+ /**
+ * The name of the LDAP attribute that stores the protocol for a Guacamole
+ * connection.
+ */
+ public static final String LDAP_ATTRIBUTE_PROTOCOL = "guacConfigProtocol";
+
+ /**
+ * The name of the LDAP attribute that stores proxy configuration
parameters
+ * for Guacamole.
+ */
+ public static final String LDAP_ATTRIBUTE_PROXY = "guacConfigProxy";
+
+ /**
+ * The proxy parameter name that configures the hostname or IP address that
+ * will be used to connect to guacd.
+ */
+ public static final String PROXY_PARAMETER_HOSTNAME = "proxy-hostname";
+
+ /**
+ * The proxy parameter name that configures the TCP port that will be used
+ * to connect to guacd.
+ */
+ public static final String PROXY_PARAMETER_PORT = "proxy-port";
+
+ /**
+ * The proxy parameter name that configures the encryption method that will
+ * be used when connected to guacd.
+ */
+ public static final String PROXY_PARAMETER_ENCRYPTION = "proxy-encryption";
Review comment:
Okay - if you've another suggestion for parsing out guacConfigProxy, I'm
certainly not opposed to a better way, but this is just the way I had gone
about implementing it - with constants for each of the possible parameter names
for `guacConfigProxy`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]