[ 
https://issues.apache.org/jira/browse/GUACAMOLE-939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17028581#comment-17028581
 ] 

Nick Couchman commented on GUACAMOLE-939:
-----------------------------------------

{quote}
Yes, docker-compose is possible, but if you put LDAP and Postgres together, I 
think guacamole.properties is much easier to handle and better documented.
{quote}

But, I think one of the points Mike is making is that what you're suggesting 
results in a very odd behavior.  If we read Docker variables from a 
guacamole.properties file, essentially what we're doing is guacamole.properties 
-> Docker variables -> guacamole.properties - we end up re-writing 
guacamole.properties with values provided already in...guacamole.properties.  
So, if you just map the guacamole.properties file location (GUACAMOLE_HOME) 
through to the container, as Mike suggested, you avoid the need for 
reading/translating/rewriting.  Yes, you still have to specify some other 
properties, like GUACD_HOSTNAME, but its pretty minimal - everything else can 
be specified directly in guacamole.properties.

> Read Docker image environment variables from guacamole.properties
> -----------------------------------------------------------------
>
>                 Key: GUACAMOLE-939
>                 URL: https://issues.apache.org/jira/browse/GUACAMOLE-939
>             Project: Guacamole
>          Issue Type: Improvement
>          Components: guacamole-docker
>    Affects Versions: 1.0.0
>            Reporter: Thomas Eggers
>            Priority: Minor
>         Attachments: start.sh
>
>
> Hi,
> I'm not a Software developer,
>  but I have linux script and docker experience.
> I startet to enhance the start.sh file in the docker image.
> This new version is pre read required variable from guacamole.properties to 
> configure the tomcat web interface. This make it much easier to start the 
> docker container without so many options.
> Other things that I like to do is:
>   - import certificate for ldaps connections
>    - docker container health status
>   - and other small things
>  Thomas



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to