Fix minor details in dockerfile

BooterMan98 / malla-interactiva

Proyecto Universitario open-source para facilitar información de las mallas en la USM.

https://booterman98.github.io/malla-interactiva/

MIT License

31 stars 31 forks source link

Fix minor details in dockerfile #14

Closed litneet64 closed 3 years ago

litneet64 commented 3 years ago

Arrregla el issue #11 e incidentemente el #12 debido a que ahora corremos el proceso dentro del container como un usuario no privilegiado. Por lo mismo, esta imagen puede ser levantada con --cap-drop=all sin problemas. Además de dejar pinneada la versión base de la imagen (puesto que será usada en producción), también arregla el puerto al correcto (8080) mostrado en las instrucciones de uso en el README y remueve terser como dependencia dentro de package.json: es requerido solo dentro del container en build-time para uso en la command-line, no dentro del código en sí.

BooterMan98 commented 3 years ago

El paquete Terser actualmente se requiere para probar cambios si no se usan contenedores dentro del proceso del desarrollo, por lo que debería seguir siendo dependencia

BooterMan98 commented 3 years ago

~~En vez de eliminarla por completo es mejor dejarla como dev dependency~~

edit: Pensandolo mejor, como también es necesario para crear el container que luego se usará en producción(Es decir, Labcomp), tiene que seguir siendo dependencia. También las otras como uglifycss.

litneet64 commented 3 years ago

Según lo conversado, finalmente el paquete terser, uglifycss y http-server no son estrictamente dependencias sino herramientas para utilizar en un workflow, por lo que acordamos incluir estos en devDependencies en package.json junto a los scripts asociados para estos workflows, los que ahora servirán tanto para uno basado solo en npm tanto como para el que construye una imagen de Docker.