geobretagne / mreport

Simple report generator with simple dataviz
GNU General Public License v3.0
10 stars 11 forks source link

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

Open abonanni opened 1 year ago

abonanni commented 1 year ago

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 :

spelhate commented 1 year ago

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

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.