MaksymBilenko / docker-oracle-12c

:whale: Docker image with Oracle Database 12c on board
Apache License 2.0
680 stars 285 forks source link

Add environment variable for ownership fix #95

Closed MatthiasLohr closed 6 years ago

MatthiasLohr commented 6 years ago

Modifying volume contents without explicit knowledge of the user can cause severe problems, even if it's only modifying file privileges. Especially for an import directory i would not expect that the script will try to modify any of my files.

If there's really a privilege problem, the user should grant the privileges on the host system, but not the docker image without the acknowledge of the user.

So, if a user really wants the script to update privileges, he should configure it (e.g. with an environment variable, see diff).

Another option (not included in this pull request) could be to give an advice to the user, how to adjust the file privileges/ownership settings if they don't match the requirements.

Best regards Matthias