SIU-Toba / framework

Framework para desarrollo rápido de aplicaciones web
http://toba.siu.edu.ar
21 stars 24 forks source link

Contemplar el uso de la variable de entorno JAVA_HOSTS para poder usar Jasper desde Docker #134

Closed tomasdelvechio closed 2 years ago

tomasdelvechio commented 2 years ago

TL;DR: Tomando en cuenta la posibilidad de que exista la variable de entorno JAVA_HOSTS, un condicional en la carga del jasper puede hacer que se pueda utilizar el mismo desde un contenedor (Se deja un ejemplo del seteo de la variable en docker-compose.yml). Cuando la variable no esta seteada, conserva el comportamiento anterior.

Nota larga: Estamos tratando de dockerizar Toba, y tuvimos alguno inconvenientes en esta parte de usar jasper como servicio fuera de localhost. Primero se intento extender toba_vista_jasperreports, pero el mismo no se puede inyectar como dependencia al servicio de impresión.

La propuesta de este PR no es la mas elegante, pero creo que no rompe otra cosa. El bloque de jasper en docker-compose.yml no funciona asi como está, pero nosotros con algunas variaciones y creando nuestra propia imagen basada en las archivadas de guarani lo pudimos hacer andar.

Si interesa, puedo proponer un yml acorde, lo que agregaría, al menos, un Dockerfile extra (el de Jasper en si).

tomasdelvechio commented 2 years ago

@enfoqueNativo Te pido disculpas, el fix de getenv subió roto, subieron corchetes en lugar de parentesis en el llamado a la función, trato de hacer un PR más en un rato

tomasdelvechio commented 2 years ago

@enfoqueNativo Te pido disculpas, el fix de getenv subió roto, subieron corchetes en lugar de parentesis en el llamado a la función, trato de hacer un PR más en un rato

Ahi ví que lo fixeaste, perdón y gracias