-
Notifications
You must be signed in to change notification settings - Fork 5.3k
Adding default manager roles in tomcat users config. #412
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
Conversation
Adding default manager roles in tomcat users config.
Adding default manager roles in tomcat users config.
|
...I guess the travis CI is currently broken |
|
For some reason
|
🤔 |
|
Hi, since the Travis CI possibly failed due to some unrelated instability, can we perhaps simply try to re-run it? |
Re-scheduled it! |
|
-1 for the above line. The "manager-jmx" role is not intended to be used by human users: it does not have CSRF protection. See
There rarely is a need to explicitly create roles like the above. When parsing the tomcat-users.xml file, all roles mentioned in users are created automatically. |
kkolinko
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The "manager-jmx" role and "manager-gui" roles should not be used together. (As I wrote is a comment earlier. Resubmitting via "Code review" panel.)
|
@kkolinko Regarding:
I wasn't aware of that. If roles are automatically generated, what's the use of the I'll remove the JMX part. |
|
Based on a PR by Arnaud Dagnelies.
Based on a PR by Arnaud Dagnelies.
Every time I install a tomcat, I first add the tomcat manager users, and I think a lot of people do this as well. However, currently, it's kind of a pain because you have to look it up first what exactly should be written and create the roles/users manually. This here makes it trivial.