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

Fix problem with root daemon not starting when sudo timeout is zero. #3669

Merged
merged 2 commits into from
Aug 19, 2024

Conversation

thallgren
Copy link
Member

The root daemon refused to start when sudo was configured with a timestamp_timeout=0. This was due to logic that first requested root privileges using a sudo call, and then relied on that these privileges were cached, so that a subsequent call using --non-interactive was guaranteed to succeed. This logic will now instead do one single sudo call, and rely solely on sudo to print an informative prompt and start the daemon in the background.

The root daemon refused to start when `sudo` was configured with a
`timestamp_timeout=0`. This was due to logic that first requested root
privileges using a sudo call, and then relied on that these privileges
were cached, so that a subsequent call using `--non-interactive` was
guaranteed to succeed. This logic will now instead do one single sudo
call, and rely solely on sudo to print an informative prompt and start
the daemon in the background.

Signed-off-by: Thomas Hallgren <[email protected]>
@thallgren thallgren added the ok to test Applied by maintainers when a PR is ready to have tests run on it label Aug 18, 2024
@github-actions github-actions bot removed the ok to test Applied by maintainers when a PR is ready to have tests run on it label Aug 18, 2024
@thallgren thallgren merged commit f31f465 into release/v2 Aug 19, 2024
8 checks passed
@thallgren thallgren deleted the thallgren/sudo-zero-timeout branch August 19, 2024 09:38
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.

Root daemon does not start (socket does not exist) Root Daemon: Not running
1 participant