[GitHub] [dolphinscheduler] caishunfeng commented on issue #12528: [Improvement] [logger-server] when config log base path out of dolphinscheduler install path, can't see task log

[GitBox <gi...@apache.org>]

caishunfeng commented on issue #12528:
URL: https://github.com/apache/dolphinscheduler/issues/12528#issuecomment-1291884874

   > org/apache/dolphinscheduler/server/log/LoggerRequestProcessor.java line:150
   > 
   > ```
   > if (path.startsWith(dsHome) && !path.contains("../") && path.endsWith(".log")) {
   >                 return true;
   >             }
   > ```
   > 
   > which force log dir can only be under dolphinscheduler install path
   
   Because it will cause a security problem, and users can read any file if no limit.


-- 
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: commits-unsubscribe@dolphinscheduler.apache.org

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