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

Update Systemd security settings #179

Merged
merged 1 commit into from
Apr 8, 2024

Conversation

rahulsundaram
Copy link

@rahulsundaram rahulsundaram commented Mar 12, 2024

As part of https://fedoraproject.org/wiki/Changes/SystemdSecurityHardening which has been approved for Fedora 40, I am working on updating Systemd services to add additional hardening settings, please review this PR and let me know if you have any feedback.

https://www.freedesktop.org/software/systemd/man/latest/systemd.exec.html has detailed information on each of these settings including the version of Systemd where they were introduced.

@t8m
Copy link
Member

t8m commented Mar 12, 2024

Hmm, do all these settings affect child processes of crond? Namely the executed cron jobs? In many cases these have high potential of breaking existing users cron jobs and are not really a good idea.

contrib/cronie.systemd Outdated Show resolved Hide resolved
contrib/cronie.systemd Show resolved Hide resolved
contrib/cronie.systemd Show resolved Hide resolved
contrib/cronie.systemd Show resolved Hide resolved
contrib/cronie.systemd Show resolved Hide resolved
contrib/cronie.systemd Outdated Show resolved Hide resolved
contrib/cronie.systemd Outdated Show resolved Hide resolved
contrib/cronie.systemd Outdated Show resolved Hide resolved
contrib/cronie.systemd Show resolved Hide resolved
contrib/cronie.systemd Outdated Show resolved Hide resolved
@t8m t8m merged commit adbd221 into cronie-crond:master Apr 8, 2024
4 checks passed
@t8m
Copy link
Member

t8m commented Apr 8, 2024

Merged after the 1.7.2 release.

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