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

[PR #4853/44e21dd4 backport][stable-4] sudoers: fix handling of state: absent (#4852) #4857

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Jun 19, 2022

This is a backport of PR #4853 as merged into main (44e21dd).

SUMMARY

This PR decouples the check for state == absent from the file existence check, so that run does not "fall through" into code paths intended for state == present.

fixes #4852

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

sudoers

ADDITIONAL INFORMATION

Refer to #4852. Please ask if anything is unclear.

* sudoers: fix handling of state: absent (#4852)

* typo fixes

(cherry picked from commit 44e21dd)
@ansibullbot
Copy link
Collaborator

cc @JonEllis @JonEllis0
click here for bot help

@ansibullbot ansibullbot added backport bug This issue/PR relates to a bug integration tests/integration module module new_contributor Help guide this first time contributor plugins plugin (any type) system tests tests labels Jun 19, 2022
@felixfontein felixfontein merged commit f8650f8 into stable-4 Jun 19, 2022
@felixfontein felixfontein deleted the patchback/backports/stable-4/44e21dd407fa86a7de78258c52871b3db1b0f187/pr-4853 branch June 19, 2022 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug integration tests/integration module module new_contributor Help guide this first time contributor plugins plugin (any type) system tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants