[GitHub] [tomcat] michael-o commented on pull request #624: Add ContextNamingInfoListener

["michael-o (via GitHub)" <gi...@apache.org>]

michael-o commented on PR #624:
URL: https://github.com/apache/tomcat/pull/624#issuecomment-1584626395

   > I think I saw this feature before in the past. No problem, although emptyOnRoot may not be very useful, so I'm not sure why this is a good idea to change things from the actual values by default.
   
   I posted this also on SO some years ago as well as with Logback's JIRA. The purpose of this flag is not to have an empty string if you try to resolve `name`. E.g., you use it for a file on the FS. `.foo`, but rather `ROOT.foo`. Same for context path.
   
   I can, of course, make the standard behavior default. It only affects root applications, though.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org