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

    
https://github.com/apache/incubator-guacamole-client/pull/11#discussion_r65133099
  
    --- Diff: guacamole-docker/README.md ---
    @@ -12,8 +12,12 @@ How to use this image
     =====================
     
     Using this image will require an existing, running Docker container with 
the
    -[guacd image](https://registry.hub.docker.com/u/glyptodon/guacd/), and 
another
    -Docker container providing either a PostgreSQL or MySQL database.
    +[guacd image](https://registry.hub.docker.com/u/glyptodon/guacd/), and 
either
    +network access to a working LDAP server, or another Docker container 
providing
    +a PostgreSQL or MySQL database.
    +
    +SQL Based Authentication
    --- End diff --
    
    Is this header necessary?
    
    I have nothing against the header itself, but it seems odd that the MySQL, 
PostgreSQL, *and* the LDAP configurations all fall beneath it.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to