problem adding two different pam entries for same module and type about puppet-augeasproviders HOT 6 CLOSED

This is a duplicate of #105

from puppet-augeasproviders.

Is this really a duplicate issue? Definitely a different problem although I suppose the fix could apply to both issues.

I tried the latest augeasproviders from git and these two entries overwrite each other even though they have a different key or path in terms of type/control/module. It seems like the unique identifier is being treated as the type/module and the control value is ignored?

from puppet-augeasproviders.

Sorry, my bad, it is a duplicate of #101, not #105.

from puppet-augeasproviders.

I know this was marked a duplicate of a now closed issue, so let me know if I should open a new issue. I tried out the latest augeasproviders (from Git earlier today as downloaded by puppet-librarian) and I am still seeing only one of the two entries I am trying to add with the pam provider using the following config. Is it possible the pam provider just doesn't let you have two entries with the same type and module (but a different control)?

pam { 'Set invalid login 3 times deny in password-auth -fail':
  ensure    => present,
  service   => 'password-auth',
  type      => 'auth',
  control   => '[default=die]',
  module    => 'pam_faillock.so',
  arguments => ['authfail','deny=3','unlock_time=604800','fail_interval=900'],
  position  => 'after *[type="auth" and module="pam_unix.so"]',
}

pam { 'Set invalid login 3 times deny in password-auth -success':
  ensure    => present,
  service   => 'password-auth',
  type      => 'auth',
  control   => 'required',
  module    => 'pam_faillock.so',
  arguments => ['authsucc','deny=3','unlock_time=604800','fail_interval=900'],
  position  => 'before *[type="auth" and module="pam_succeed_if.so"]',
}

I am getting one of the two entries like the following and it is at the end of the password-auth file.

auth        required      pam_env.so
auth        sufficient    pam_unix.so nullok try_first_pass
auth    [default=die]   pam_faillock.so authfail        deny=3  unlock_time=604800      fail_interval=900
auth    requisite       pam_succeed_if.so       uid     >=      500     quiet
auth    required        pam_tally.so    deny=3
auth        required     pam_deny.so

If I comment out the first puppet "pam" and run again then it shows the other entry (see below). At one point I had the position attribute on the entry that wasn't showing up referencing a module/type combination that didn't exist so the entry was going to the end of the password-auth file and was being overwritten there by the entry that was actually showing up.

auth        required      pam_env.so
auth        sufficient    pam_unix.so nullok try_first_pass
auth    required        pam_faillock.so authsucc        deny=3  unlock_time=604800   fail_interval=900
auth    requisite       pam_succeed_if.so       uid     >=      500     quiet
auth    required        pam_tally.so    deny=3
auth        required     pam_deny.so

from puppet-augeasproviders.

Indeed, control is a property, not a parameter, so it is meant to converge for a given combination of service and type. If you think this is a bug, please open a new ticket to request control to be used as a parameter instead.

from puppet-augeasproviders.

Issue #114 deals with this now.

from puppet-augeasproviders.