Skip to content
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

cron 环境变量转义 & 新的环境变量 Ray_Crontab #256

Merged
merged 6 commits into from
Aug 23, 2022
Merged

cron 环境变量转义 & 新的环境变量 Ray_Crontab #256

merged 6 commits into from
Aug 23, 2022

Conversation

syrinka
Copy link
Contributor

@syrinka syrinka commented Aug 22, 2022

【内容】:
请描述您将贡献的内容

  • fix: cron 加载环境变量未对特殊字符进行转义
  • feat: 环境变量 Ray_Crontab,替代 my_crontab 文件的功能,已补充对应文档

@syrinka syrinka changed the title Develop cron 环境变量转义 & 新的环境变量 Ray_Crontab Aug 22, 2022
docker/entry.sh Outdated Show resolved Hide resolved
@RayWangQvQ
Copy link
Owner

请教个问题,之前的环境变量是写到/etc/environment里,现在改成cron文件里主动"BASH_ENV=/etc/cron.env",这样改是有什么原因吗?

@syrinka
Copy link
Contributor Author

syrinka commented Aug 22, 2022

请教个问题,之前的环境变量是写到/etc/environment里,现在改成cron文件里主动"BASH_ENV=/etc/cron.env",这样改是有什么原因吗?

原来的写法无法正确处理一些字符,比如 # 和换行符等,解析到的环境变量可能与实际传入的不一致

ref:

@RayWangQvQ RayWangQvQ merged commit d775ca8 into RayWangQvQ:develop Aug 23, 2022
@RayWangQvQ
Copy link
Owner

Merged, these update will be included in the next release version, thx for your nice work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants