[GitHub] [dolphinscheduler] zhuangchong commented on a diff in pull request #12909: [Bug-12908][Worker]Fix execute shell task exception no dolphinscheduler_env.sh file execute permission

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

zhuangchong commented on code in PR #12909:
URL: https://github.com/apache/dolphinscheduler/pull/12909#discussion_r1023408308


##########
dolphinscheduler-worker/src/main/bin/start.sh:
##########
@@ -21,7 +21,6 @@ DOLPHINSCHEDULER_HOME=${DOLPHINSCHEDULER_HOME:-$(cd $BIN_DIR/..; pwd)}
 
 source "$DOLPHINSCHEDULER_HOME/conf/dolphinscheduler_env.sh"
 
-chmod -R 700 ${DOLPHINSCHEDULER_HOME}/conf

Review Comment:
   There is account password information in the application.yaml file in the conf directory, so it is not recommended to set the permission to 755.
   
   If it is set to `chmod -R 755 ${DOLPHINSCHEDULER_HOME}/conf/dolphinscheduler_env.sh`, when the conf folder does not have other user execution permissions, it will still report that the dolphinscheduler_env.sh file permissions are insufficient.
   
   Do you have any good suggestions for only setting dolphinscheduler_env.sh file permissions to 755 and other file permissions to 700?



-- 
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