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

Multiple Users #5

Closed
alexandcote opened this issue Jul 10, 2016 · 2 comments
Closed

Multiple Users #5

alexandcote opened this issue Jul 10, 2016 · 2 comments

Comments

@alexandcote
Copy link
Contributor

As-tu une idée comment nous pourrions implémenter le support de plusieurs users ?

Je suggère d'utilisé un array de user dans le fichier de configuration comme ceci:

htpasswd:
      - name: firstUser
        password: firstPassword
      - name: secondUser
        password: secondPassword

Par contre, lorsque j'essaye de l'implémenter dans ton role, je n'y parvient pas.

- name: Set htpasswd
  htpasswd:
    path: "{{ htpasswd_path }}/{{ item.0.key }}"
    name: "{{ item.1.name }}"
    password: "{{ item.1.password }}"
    crypt_scheme: "{{ item.1.crypt|default(omit) }}"
    owner: root
    group: root
    mode: 0644
  with_subelements:
    - wordpress_sites
    - htpasswd
    - skip_missing: yes

En gros, item.0.key n'existe pas. Je ne sais pas si tu as une autre solution ?

Thanks.

@alexandcote alexandcote changed the title Multiple User Multiple Users Jul 10, 2016
@louim
Copy link
Owner

louim commented Jul 11, 2016

Je viens de regarder ça, le problème c'est que quand tu itère sur un hash ( wordpress_sites), avec with_subelements, tu perds la propriété key, parce qu'il y accède comme un array, donc item.0 c'est le contenu d'une key. Il n'y a pas vraiment de manière facile d'accéder à la key rendu là, parce que tu es déjà dans l'itération.

Cependant, étant donné qu'on s'en sert seulement pour nommer le path vers le fichier, on pourrait utiliser item.0.site_hosts.0 au lieu de item.0.key (pas testé). C'est moins clean, mais au moins c'est une des valeur représentative qui risque le moins de changer.

@louim
Copy link
Owner

louim commented Oct 1, 2016

Fixed in v1.1.0

@louim louim closed this as completed Oct 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants