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

Dockerisation - template frontend/templates/index.html - interprétation de variables #189

Open
abonanni opened this issue Oct 12, 2022 · 1 comment

Comments

@abonanni
Copy link

La configuration docker est incomplète et ne permet pas de faire fonctionner le moteur de template jinja2 sur la partie frontend correctement.

En effet, si l'on analyse la configuration nginx, on remarque que le fichier "frontend/templates/index.html" est servi directement :

    location /mreport {	
        alias /var/www/reports;
        index /templates/index.html;
        location ~ config\.json {
            try_files $uri =404;
        }
        try_files $uri /templates/index.html;
    }

Toutes les balises de templates comme {{ options|tojson }} ne seront donc jamais interprétées.

Plusieurs solutions possibles :

  • modifier le conteneur backend afin de servir via gnunicorn aussi bien le dossier frontend que le dossier backend
  • ajout d'un second conteneur fonctionnant avec gunicorn afin de servir le dossier frontend ; et renommer le conteneur frontend en webserver
@spelhate
Copy link
Collaborator

Bonjour,
A l'origine du projet, sans docker, il y avait deux façons différentes de servir le front:

  • via python
  • via une configuration apache2 ou nginx.

Effectivement, avec la dernière solution, il n'est pas possible d'utiliser les balises jinja2. Et c'est cette dernière qui a été intégrée dans la conf docker.

Pour info, voici détaillées les deux façons de servir le front

Les deux solutions que vous proposez me semblent intéressantes. Ca laisse le choix à l'utilisateur d'utiliser Docker ou non.
Surtout que pour gérer les routes avec Nginx ou Apache, les règles ne sont pas toujours simples.

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