Skip to content

Get rid of tomcat user for ease of use #6

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
Dec 1, 2014
Merged

Get rid of tomcat user for ease of use #6

merged 1 commit into from
Dec 1, 2014

Conversation

yosifkit
Copy link
Member

I have tags 8-jre8 and 8-jre7/latest building in an automated build if someone can test them to ensure that their code works in it.

@bryantrobbins, or @jgangemi 🙏, think you could give it a quick test when you have a moment, especially in regards to our discussion about mounting into $CATALINA_HOME/webapps in #3.

@bryantrobbins
Copy link

Tested with both yosifkit/tomcat:8-jre8 and yosifkit/tomcat:latest and was able to:
-Mount a war into /usr/local/tomcat/webapps
-Mount a custom tomcat-users.xml into /usr/local/tomcat/conf to enable access to the /manager app

docker run -dit -p 8080:8080 --name test_tomcat -v /tmp/mytomcat/tomcat-users.xml:/usr/local/tomcat/conf/tomcat-users.xml -v /tmp/mytomcat/webapps/example.war:/usr/local/tomcat/webapps/example.war yosifkit/tomcat:8-jre8

Happy to write/share examples into docs if needed.

Cheers,
...Bryan

@yosifkit
Copy link
Member Author

yosifkit commented Dec 1, 2014

Thanks @bryantrobbins. Ping @tianon.

@tianon
Copy link
Member

tianon commented Dec 1, 2014

LGTM

tianon added a commit that referenced this pull request Dec 1, 2014
Get rid of tomcat user for ease of use
@tianon tianon merged commit 116d025 into docker-library:master Dec 1, 2014
@tianon tianon deleted the root branch December 1, 2014 16:55
tianon added a commit to infosiftr/stackbrew that referenced this pull request Dec 1, 2014
- `java`: security updates (docker-library/openjdk#18)
- `mysql`: 5.5.41 and 5.6.22
- `node`: NPM 2.1.10
- `tomcat`: switch to root user (docker-library/tomcat#6)
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.

5 participants