Grunt server écoute sur 0.0.0.0

Zenika-Training / zenika-formation-framework

Tools to build and present our training material

8 stars 25 forks source link

Grunt server écoute sur 0.0.0.0 #90

Closed gquintana closed 7 years ago

gquintana commented 9 years ago

Cela signifie que pendant la formation le stagiaire un peu débrouillard est capable d'aspirer les slides depuis sa machine, sans qu'on s'en rende compte.

Il vaudrait mieux écouter sur 127.0.0.1.

ggerbaud commented 9 years ago

C'est nécessaire pour Docker.

Pour une formation, utiliser le version sur app-engine. Ou la version local avec Grunt en changeant le gruntfile localement (0.0.0.0 -> 127.0.0.1)

ldez commented 9 years ago

il est possible dans une futur version de faire un goal dédié pour cela, genre : grunt slide (ou plutôt gulp slide ^^) qui changera l'IP.

gquintana commented 9 years ago

+1 Pour le goal dédié. On pourrait désactiver le watch/livereload pendant la formation (ca pompe des ressources pour rien).

Pour l'IP, on pourrait aller chercher une variable d'environnement (pour Docker c'est le plus simple il me semble) qui contiendrait l'IP à binder, quitte à prendre 127.0.0.1 si elle n'est pas définie. Sinon ne peut-on pas externaliser cette config dans fichier au niveau du projet/formation?

On ne peut pas forcément utiliser la version app-engine (formation custom, pas d'internet ou accès internet pourri...)

ldez commented 9 years ago

plusieurs points :

utiliser une variable d'environnement pourquoi pas mais ça me semble moins simple que faire un goal dédié.
pour le coté externalisé, même idée, un goal dédié bien pensé devrait résoudre le problème.
pour la CI, elle produit aussi des PDF et tu peut aussi produire ces PDF en local ce qui résout le problème d'aspiration via le réseau.

gquintana commented 9 years ago

Pas compris ton dernier point.

ldez commented 9 years ago

Tu peut récupérer le PDF des slides produit par la CI pour une formation "standard" et si tu fais une formation custom, tu peux produire en local le PDF des slides. Après il suffit d'avoir un lecteur de PDF et comme ça tu n'as pas besoin de lancer Node.

gquintana commented 9 years ago

C'est un peu dommage d'utiliser le PDF pour animer la formation: notes, navigation... Cela dit, je pourrais arrêter le serveur une fois les slides chargés dans le browser.

ldez commented 9 years ago

Je suis d'accord. Pour l'ip, il n'y a pas de gros problème. Par contre pour le liveload, le nouveau système devrait être moins consommateur normalement, mais il va être plus difficile à couper. Pour moi le liveload c'est plus un avantage car ça permet de corriger les erreurs ou d'ajouter des notes en live.

Je regarderai comment faire au mieux.

jlandure commented 8 years ago

Si j'essaie de résumer, nous avons plusieurs façons d'animer la formation:

en livereload avec gulp/grunt sur 0.0.0.0 (non protégé face aux attaques des méchants stagiaires)
en web avec les sites sur appspot.com
utiliser les PDFs (pas d'issue pour conserver les notes dans les pdf ? ^^)
en site statique classique si on télécharge le zip produit avec #97 (en nginx, node, python, docker) sur 127.0.0.1
avec Docker sur 0.0.0.0 (non protégé face aux attaques des méchants stagiaires)
proposé ici: gulp slides qui ferait le "package" puis servirait statiquement tout ça sans livereload sur 127.0.0.1

Cela signifie que nous aurions un mode "edition" (0.0.0.0) VS un mode "presentation" (127.0.0.1) c'est bien ça que l'on souhaite ?

ldez commented 8 years ago

pas de mode édition mais 2 modes présentations : privé / public

jlandure commented 7 years ago

Vu qu'on va filer les supports, je propose de faire simple et de laisser "ouvert" sur 0.0.0.0

hgwood commented 7 years ago

@jlandure exact. A rouvrir si cette hypothèse est incorrecte.