benrubson commented on a change in pull request #554:
URL: https://github.com/apache/guacamole-client/pull/554#discussion_r532914265
##########
File path: guacamole-docker/bin/start.sh
##########
@@ -659,9 +659,15 @@ associate_header() {
##
start_guacamole() {
+ # User-only writable CATALINA_BASE
+ export CATALINA_BASE=$GUACAMOLE_HOME/tomcat
+ for dir in logs temp webapps work; do
+ mkdir -p $CATALINA_BASE/$dir
+ done
+ cp -R /usr/local/tomcat/conf $CATALINA_BASE
Review comment:
Well, it would work.
But at least one other PR plans to modify the configuration : #489.
Let's then keep it writable 👍
----------------------------------------------------------------
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]
