[
https://issues.apache.org/jira/browse/GUACAMOLE-41?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15311172#comment-15311172
]
ASF GitHub Bot commented on GUACAMOLE-41:
-----------------------------------------
Github user mike-jumper commented on a diff in the pull request:
https://github.com/apache/incubator-guacamole-client/pull/11#discussion_r65447441
--- Diff: guacamole-docker/README.md ---
@@ -129,17 +126,17 @@ Deploying Aoache Guacamole with LDAP authentication
-e LDAP_CONFIG_BASE_DN=ou=connections,dc=example,dc=com \
-d -p 8080:8080 glyptodon/guacamole
-Linking Guacamole to your LDAP directory will require additional
configuration parameters
+Using Guacamole with your LDAP directory will require additional
configuration parameters
specified via environment variables. These variables collectively describe
how Guacamole
will connect to LDAP:
-1. `LDAP_HOSTNAME` - The base of the DN for all Guacamole users.
+1. `LDAP_HOSTNAME` - The hostname or IP address of your LDAP server.
2. `LDAP_USER_BASE_DN` - The base of the DN for all Guacamole users.
3. `LDAP_PORT` - The port your LDAP server listens on. (Optional)
--- End diff --
If you think that's better, then cool. I'm more of the mind that separate
would be easier to read, but I don't think this way is bad in itself.
> Document how to use LDAP with Docker
> ------------------------------------
>
> Key: GUACAMOLE-41
> URL: https://issues.apache.org/jira/browse/GUACAMOLE-41
> Project: Guacamole
> Issue Type: Improvement
> Components: guacamole-docker
> Affects Versions: 0.9.9
> Reporter: James Johnston
>
> As part of GUAC-1467 on the Glyptodon JIRA, the Docker image for guacamole
> was improved to allow specification of LDAP authentication parameters.
> However, as an end-user, there is no way for me to even know this
> functionality is available because it is not documented at all. It appears to
> be undocumented.
> I think the README.md at
> https://github.com/glyptodon/guacamole-docker/blob/master/README.md should be
> updated with information about how to use LDAP authentication, so that people
> can use this new feature. :)
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
