saltstack-formulas / users-formula

Configure users via pillar
http://docs.saltstack.com/en/latest/topics/development/conventions/formulas.html
Other
99 stars 360 forks source link

fixed insertion of multiple authorized keys via ssh_auth_pillar #156

Closed qno closed 7 years ago

qno commented 7 years ago

The creation of ssh authorized_keys containing multiple keys is broken if contents_pillar option is used. It only used the first item in list. I "reverted" it to use the content option, which works now.

aboe76 commented 7 years ago

can somebody confirm this? I don't have a setup ready with multiple keys per user? @0xf10e do you?

EvaSDK commented 7 years ago

According to documentation contents_pillar takes a single pillar value. Would have to check the module to be absolutely sure but this seems like a logical change.

0xf10e commented 7 years ago

I don't know how up2date the checkout(s) of this formula I use at work* is(/are)… I can't remember issues like that but I wouldn't think this change to cause any troubles.

(* on vacation right now ;))

aboe76 commented 7 years ago

thanks for this