Cadoles/ansible-role-sso
10
0
Fork 0
Code Issues 1 Pull Requests 1 Projects Releases Wiki Activity

Ajout variables d'environement hydra-oidc #3

Merged
wpetit merged 3 commits from hydra_oidc_options into master 2022-09-14 12:11:01 +02:00
Conversation 0 Commits 3 Files Changed 2 +3 -3
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit a3c83fbd86 - Show all commits

2
defaults/main.yml
View File

@ -199,7 +199,7 @@ hydra_oidc_post_logout_redirect_url:
hydra_oidc_scope: openid email
hydra_oidc_client_id:
hydra_oidc_client_secret:
hydra_oidc_environment_opts: []
hydra_oidc_additionnal_env: {}
wpetit marked this conversation as resolved Outdated
wpetit commented 2022-09-13 11:02:23 +02:00
Outdated
Review
Copy Link

J'aurais nommé la variable différement: hydra_oidc_additionnal_env. Et par défaut, je préfererais que ce soit un dictionnaire (un tableau clé->valeur), valeur par défaut {}.

De cette manière, on pourra définir les variables d'environnement additionnelles de la manière suivante:

hydra_oidc_additionnal_env:
    MY_ENV_VAR: 'bar'
    MY_SECOND_ENV_VAR: 'foo'
J'aurais nommé la variable différement: `hydra_oidc_additionnal_env`. Et par défaut, je préfererais que ce soit un dictionnaire (un tableau clé->valeur), valeur par défaut `{}`. De cette manière, on pourra définir les variables d'environnement additionnelles de la manière suivante: ```yaml hydra_oidc_additionnal_env: MY_ENV_VAR: 'bar' MY_SECOND_ENV_VAR: 'foo' ```
hydra_oidc_attributes_rewrite_rules:
email:

4
templates/cadoles-pod-hydra-oidc-v1.conf.j2
View File

@ -19,7 +19,7 @@ PODMAN_ARGS="\
-e 'CLIENT_ID_FC={{ hydra_oidc_client_id }}' \
-e 'CLIENT_SECRET_FC={{ hydra_oidc_client_secret }}' \
-e 'TRUSTED_PROXIES=127.0.0.1,10.0.2.0/24' \
{% for item in hydra_oidc_environment_opts %}
-e '{{ item.environment_var_name }}={{ item.value }}' \
{% for item in hydra_oidc_additionnal_env.values() %}
wpetit marked this conversation as resolved Outdated
wpetit commented 2022-09-13 11:04:19 +02:00
Outdated
Review
Copy Link

Si on bascule sur un dictionnaire (cf. commentaire précédent), la boucle serait plutôt:

{% for item in hydra_oidc_additionnal_env.values() %}
	-e '{{ item.name }}={{ item.value }}' \
{% endfor %}
Si on bascule sur un dictionnaire (cf. commentaire précédent), la boucle serait plutôt: ``` {% for item in hydra_oidc_additionnal_env.values() %} -e '{{ item.name }}={{ item.value }}' \ {% endfor %} ```
wpetit commented 2022-09-14 09:49:29 +02:00
Outdated
Review
Copy Link

Il ne faudrait pas un \ à la fin de la ligne ?

Il ne faudrait pas un `\` à la fin de la ligne ?
mlamalle commented 2022-09-14 11:35:10 +02:00
Outdated
Review
Copy Link

A tester, mais j'en doute vue que c'est une construction de commande bash, et qu'on est sur l'itérateur jinja là.

A moins que ce fameux itérateur crée une ligne vide.

A tester, mais j'en doute vue que c'est une construction de commande bash, et qu'on est sur l'itérateur jinja là. A moins que ce fameux itérateur crée une ligne vide.
-e '{{ item.name }}={{ item.value }}' \
{% endfor %}
"