Skip to content
This repository has been archived by the owner on Dec 22, 2023. It is now read-only.

MUMMNG-1681 : Dockerfile #18

Merged
merged 3 commits into from
Jul 8, 2015
Merged
Show file tree
Hide file tree
Changes from 2 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions Dockerfile
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
FROM tomcat:8-jre7
MAINTAINER Tim Levett <tim.levett@wisc.edu>

ADD ./my-profile-webapp/target/profile /usr/local/tomcat/webapps/profile

ADD tomcat-users.xml /usr/local/tomcat/conf/
7 changes: 7 additions & 0 deletions tomcat-users.xml
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
<?xml version='1.0' encoding='utf-8'?>
<tomcat-users>
<role rolename="manager-gui"/>
<role rolename="manager-gui"/>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two <role elements with the same rolename?

<role rolename="manager-script"/>
<user username="admin" password="admin" roles="manager,manager-gui,manager-script" />
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What blocks something other than the intended build script from using this Tomcat manager username and password to deploy something untoward to the running Docker instance?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was thinking of removing this, was just for dev

</tomcat-users>