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 +4
cmsassot commented 2022-09-13 10:46:13 +02:00
Member
Copy Link
No description provided.
cmsassot added 1 commit 2022-09-13 10:46:22 +02:00
wpetit requested changes 2022-09-13 11:06:02 +02:00
defaults/main.yml Outdated
@ -199,6 +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: []
wpetit commented 2022-09-13 11:02:23 +02:00
Owner
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' ```
wpetit marked this conversation as resolved
templates/cadoles-pod-hydra-oidc-v1.conf.j2 Outdated
@ -19,4 +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 %}
wpetit commented 2022-09-13 11:04:19 +02:00
Owner
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 marked this conversation as resolved
cmsassot added 1 commit 2022-09-13 11:15:47 +02:00
wpetit requested changes 2022-09-14 09:49:52 +02:00
templates/cadoles-pod-hydra-oidc-v1.conf.j2 Outdated
@ -19,4 +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_additionnal_env.values() %}
wpetit commented 2022-09-14 09:49:29 +02:00
Owner
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
Owner
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.
wpetit marked this conversation as resolved
cmsassot added 1 commit 2022-09-14 11:27:15 +02:00
wpetit merged commit 3b7b187a60 into master 2022-09-14 12:11:01 +02:00
wpetit deleted branch hydra_oidc_options 2022-09-14 12:11:04 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
wpetit
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Cadoles/ansible-role-sso#3
No description provided.
Delete Branch "hydra_oidc_options"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?