[GitHub] wohali commented on a change in pull request #110: Conditionally set ownership and permissions in entrypoint

[GitBox <gi...@apache.org>]

wohali commented on a change in pull request #110: Conditionally set ownership and permissions in entrypoint
URL: https://github.com/apache/couchdb-docker/pull/110#discussion_r226799332
 
 

 ##########
 File path: 2.2.0/docker-entrypoint.sh
 ##########
 @@ -25,13 +25,26 @@ if [ "$1" = 'couchdb' ]; then
 fi
 
 if [ "$1" = '/opt/couchdb/bin/couchdb' ]; then
-	# we need to set the permissions here because docker mounts volumes as root
-	chown -fR couchdb:couchdb /opt/couchdb || true
+	# Check that we own everything in /opt/couchdb and fix if necessary
+	find /opt/couchdb \! \( -user couchdb -group couchdb \) -exec chown couchdb:couchdb '{}' +
 
 Review comment:
   Needs a `-f`

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services