Skip to content
This repository has been archived by the owner on Dec 26, 2020. It is now read-only.

Added support for TrustedUserCAKeys and AuthorizedPrincipalsFile. #157

Merged
merged 3 commits into from
Apr 17, 2018
Merged

Added support for TrustedUserCAKeys and AuthorizedPrincipalsFile. #157

merged 3 commits into from
Apr 17, 2018

Conversation

gdelafond
Copy link

This PR allow the configuration for TrustedUserCAKeys and AuthorizedPrincipalsFile globally.

It will also write trusted certificate authorities public keys and principals if defined in the file defined by ssh_trusted_user_ca_keys_file and by ssh_authorized_principals.path.

The format for the principals hash is :

  • path: the path of the file that will contains principals
  • principals: a list of principals
  • owner: the file owner (defaults to ssh_owner)
  • group: the file group (defaults to ssh_group)
  • directoryowner: the file parent directory owner (defaults to ssh_owner)
  • directorygroup: the file parent directory group (defaults to ssh_group)

@rndmh3ro
Copy link
Member

Thanks @gdelafond, this looks good!

Two things:

  • can you put the new tasks into a separate tasks-file and include it from the main.yml (with a single when-clause at the include). I plan on separating more tasks from the main.yml to get some more structure.
  • can you please add the examples of the variables next to the veriables in the defaults file (like its done for the ssh_remote_hosts variable)?

@gdelafond
Copy link
Author

@rndmh3ro I made the modifications, does it looks good ?

@rndmh3ro rndmh3ro merged commit 4b2ffd9 into dev-sec:master Apr 17, 2018
@rndmh3ro
Copy link
Member

Looks great, thank you!

@gdelafond gdelafond deleted the add-trusted-user-ca-keys-and-authorized_principals branch April 17, 2018 07:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants