Skip to content

feat: add umask configuration support #4972

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

wanglu-cell
Copy link

Which issue(s) this PR fixes:
Fixes #

What this PR does / why we need it:
增加 umask 配置支持,使日志文件权限控制更灵活。当用户需要指定非默认文件权限时,可通过配置 umask 参数实现。
Docs Changes:

Release Note:
新增功能: 支持通过 umask 参数控制输出文件权限

@daipom
Copy link
Contributor

daipom commented May 21, 2025

Thanks for your contribution!

Could you please clarify what this PR does?

Fluentd already has umask command line option.

The limitation of this option is:

  • There is no system config option yet.
  • It works only when using --daemon or --no-supervisor.
    • For example, it does not work when using Docker.

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