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.

Sign up for GitHub

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

Jump to bottom

chatmaild: fix umask for doveauth + metadata #494

Merged
merged 3 commits into from
Feb 17, 2025
Merged

chatmaild: fix umask for doveauth + metadata #494

merged 3 commits into from
Feb 17, 2025

Conversation

missytake
Copy link
Contributor

fix #453

@missytake missytake requested a review from link2xt February 17, 2025 00:35
@missytake
Copy link
Contributor Author

Guess what, the CI was only half-flaky. It was also very broken xD I should have followed my intuition that with the umask command you wouldn't say 0700 but 0077, I thought maybe systemd uses the reasonable notation. But well, when I saw that new users aren't created because /run/doveauth/doveauth.socket has ---rwxrwx permissions I suspected what was wrong

@missytake missytake merged commit f4e3716 into main Feb 17, 2025
6 checks passed
@missytake missytake deleted the umask branch February 17, 2025 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adbenitez adbenitez adbenitez approved these changes

@link2xt link2xt Awaiting requested review from link2xt

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Set UMask for metadata and doveauth systemd units to 0700
2 participants
@missytake @adbenitez