Extension des instances possibles à stylo-dev #672
Conversation
✅ Deploy Preview for stylo-dev ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
| @@ -58,6 +58,7 @@ services: | |||
| image: "davidbgk/stylo-export:0.0.3" | |||
| environment: | |||
| - SE_PANDOC_API_BASE_URL=http://pandoc-api:8000/latest/ | |||
| - SE_ALLOWED_INSTANCE_DOMAINS=["stylo.huma-num.fr", "stylo-dev.huma-num.fr"] | |||
There was a problem hiding this comment.
j'ai fusionné trop vite, effectivement ça ne va pas fonctionner, les variables d'environnements sont des chaines de caractères uniquement.
SE_ALLOWED_INSTANCE_DOMAINS="stylo.huma-num.fr,stylo-dev.huma-num.fr"
et ensuite dans le code il faut faire un split pour construire le tableau
There was a problem hiding this comment.
mmh je vois le souci, une option semble être de passer '["foo", "bar"]' pour pouvoir faire un json.loads() dessus mais ça ne résout pas la dualité avec le fait de pouvoir aussi déclarer une liste Python dans la config.
Un truc moche serait de faire json.loads(str(["foo", "bar"]).replace("'", '"')) dans le code ensuite 😬
>>> from_env = '["foo", "bar"]'
>>> json.loads(str(from_env).replace("'", '"'))
['foo', 'bar']
>>> from_python = ["foo", "bar"]
>>> json.loads(str(from_python).replace("'", '"'))
['foo', 'bar']C'est pas super propre mais ça a le mérite d'avoir la même fonction pour récupérer la liste. Qu'en penses-tu ?
There was a problem hiding this comment.
Pourquoi pas... il n'est pas possible de faire la distinction entre une valeur configurée dans la config Python et une valeur qui provient d'une variable d'environnement?
L'autre solution c'est de déclarer une chaîne de caractères dans la config Python.
À toi de voir ce que tu préfères j'ai pas vraiment un opinion tranché sur le sujet 😄
This reverts commit 0544091.
No description provided.