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

Add Ansible remediation to sssd_enable_pam_services #11796

Merged
merged 9 commits into from
Apr 15, 2024

Commits on Apr 5, 2024

  1. Add Ansible remediation to sssd_enable_pam_services

    This commit adds an Ansible remediation to rule sssd_enable_pam_services.
    
    Fixes: ComplianceAsCode#11753
    jan-cerny committed Apr 5, 2024
    Configuration menu
    Copy the full SHA
    f946dd3 View commit details
    Browse the repository at this point in the history

Commits on Apr 9, 2024

  1. Configuration menu
    Copy the full SHA
    6c6423c View commit details
    Browse the repository at this point in the history
  2. Improve regex

    Imrpove the regular expression so that it won't match entries
    containing the pam entry. This would lead to duplication
    of pam entries if the Playbook is executed twice or multiple times.
    jan-cerny committed Apr 9, 2024
    Configuration menu
    Copy the full SHA
    5926b17 View commit details
    Browse the repository at this point in the history
  3. Fix unused variables

    jan-cerny committed Apr 9, 2024
    Configuration menu
    Copy the full SHA
    5be46e6 View commit details
    Browse the repository at this point in the history
  4. Add test scenario for conf.d directory

    This test scenario tests a pass situation if the conf.d directory
    is used for configuration.
    jan-cerny committed Apr 9, 2024
    Configuration menu
    Copy the full SHA
    d0ae92f View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    a5fc104 View commit details
    Browse the repository at this point in the history

Commits on Apr 10, 2024

  1. Stop using FQMN for ini_file

    Resolves this fail on CentOS 7:
    
    34/44 Test #34: ansible-playbook-syntax-check-rhel7 ..............................***Failed    2.69 sec
    [WARNING]: provided hosts list is empty, only localhost is available. Note that
    the implicit localhost does not match 'all'
    ERROR! couldn't resolve module/action 'community.general.ini_file'. This often indicates a misspelling, missing collection, or incorrect module path.
    jan-cerny committed Apr 10, 2024
    Configuration menu
    Copy the full SHA
    2dc0247 View commit details
    Browse the repository at this point in the history

Commits on Apr 12, 2024

  1. Prevent removing existing entries

    If a services key exists, and contains a compliant line in sssd.conf
    which also contains other services, eg. `services = nss,pam`
    we shouldn't remove the other services but we should keep them.
    jan-cerny committed Apr 12, 2024
    Configuration menu
    Copy the full SHA
    4592d2e View commit details
    Browse the repository at this point in the history

Commits on Apr 15, 2024

  1. Update task to check if line is already present

    The task was updated to ensure that last task is only executed when
    there isn't already any definition of "services" in sssd section. Only
    this case a new line will be included. This is to avoid removing
    existing options from existing configuration.
    marcusburghardt authored and jan-cerny committed Apr 15, 2024
    Configuration menu
    Copy the full SHA
    1f7b478 View commit details
    Browse the repository at this point in the history