Skip to content

Adjust permissions further for arbitrary user even in the face of plugins with Gemfiles #142

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Nov 20, 2018

Conversation

tianon
Copy link
Member

@tianon tianon commented Nov 20, 2018

Builds on #136, #141

Fixes #137

Tested (successfully) via:

$ mkdir oh-no
$ echo "gem 'l'" > oh-no/Gemfile
$ docker run -it --rm -v "$PWD/oh-no":/usr/src/redmine/plugins/oh-no:ro redmine:3.4
$ docker run -it --rm -v "$PWD/oh-no":/usr/src/redmine/plugins/oh-no:ro --user 1000:1000 redmine:3.4

@yosifkit yosifkit merged commit c19e8e1 into docker-library:master Nov 20, 2018
@yosifkit yosifkit deleted the aribtrary-user-again branch November 20, 2018 22:07
tianon added a commit to infosiftr/stackbrew that referenced this pull request Nov 20, 2018
- `elasticsearch`: 6.5.1
- `ghost`: 2.6.1
- `irssi`: `gpg --batch` (jessfraz/irssi#21)
- `joomla`: joomla-docker/docker-joomla#69
- `kibana`: 6.5.1
- `logstash`: 6.5.1
- `mariadb`: 10.3.11
- `matomo`: 3.7.0
- `openjdk`: 12-ea+20, windowsservercore-1803 for jdk8
- `redmine`: arbitrary user fixes (docker-library/redmine#141, docker-library/redmine#142)
- `tomcat`: 7.0.92
autophagy pushed a commit to crate/official-images that referenced this pull request Dec 12, 2018
- `elasticsearch`: 6.5.1
- `ghost`: 2.6.1
- `irssi`: `gpg --batch` (jessfraz/irssi#21)
- `joomla`: joomla-docker/docker-joomla#69
- `kibana`: 6.5.1
- `logstash`: 6.5.1
- `mariadb`: 10.3.11
- `matomo`: 3.7.0
- `openjdk`: 12-ea+20, windowsservercore-1803 for jdk8
- `redmine`: arbitrary user fixes (docker-library/redmine#141, docker-library/redmine#142)
- `tomcat`: 7.0.92
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants