Duplicated MFAPasswordMaxDays entry in bastion.conf

[axl89]

While doing a Chef cookbook to manage this awesome software, I found out that the MFAPasswordMaxDays entry is duplicated in the bastion.conf.dist file in v3.01.00:

# MFAPasswordMaxDays (int >= 0)
#    DESC: For the PAM UNIX password MFA, sets the maximum amount of days after which the password must be changed (see `chage -M')
# DEFAULT: 90
"MFAPasswordMaxDays": 90,
#
# MFAPasswordMaxDays (int >= 0)
#    DESC: For the PAM UNIX password MFA, sets the number of days before expiration on which the user will be warned to change his password (see `chage -W')
# DEFAULT: 15
"MFAPasswordMaxDays": 15,

I'm not sure which one you'd like to keep, so I can't make a PR, but I hope this issue is useful! 😄

[Alkorin]

Thanks for the report!

It's not a duplicate entry but a wrong config name.

[speed47]

Side note @axl89, we have a Puppet module out since a few days, because that's what we happen to be using internally ;)
So that's cool you're working on a Chef cookbook!

[axl89]

Thanks, @speed47 ! I'll follow your work closely, I really like this project and we will start using it very soon in our production environments. I aim to maintain the cookbook up to date with your new changes and configurations! 😄

[speed47]

I'll try your cookbook when I find some time. We might want to reference it in the readme as a third party tool at some point!