https://issues.apache.org/bugzilla/show_bug.cgi?id=57455
--- Comment #1 from Mark Thomas <ma...@apache.org> --- Thanks for the patch. I can confirm that it fixes the issues described. But... This has the potential to get messy. Neither " nor , is permitted in a file name in Windows but both are acceptable in unix and friends. We started off simply using ',' (comma) as a delimiter. This assumed ',' never appeared in the value. Then we had a bug report that paths containing ',' broke things. To fix this we added optional quoting using '"'. This allowed ',' to appear in the value if the value was quoted. This assumed '"' never appeared in a value. Now we have this bug. I am concerned that the proposed solution is a sticking plaster rather than a complete fix and that in a few months time we'll have someone raising a bug about a different edge case (I can already see several valid file names that will break this). Unless we switch to using an escape character of some form, we are always going to have these issues. And I'm not at all sure that escaping will be possible since the values being parsed are obtained - in the default case - via unescaped environment variables. I'm leaning towards 'fixing' this by adding a comment that paths with '"' are not permitted to catalina.propeties. -- You are receiving this mail because: You are the assignee for the bug. --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
