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

Implementando estructura de menú dinámicos #68

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

jselvamadrigal
Copy link
Collaborator

Se realiza una limpieza de código comentado, se elimina código de splash screen no utilizado, se mejora la asignación del look and feel. Por ultimo se cambia el nombre de la clase MiAplicacionPICO por AplicacionPICO.

@jselvamadrigal jselvamadrigal marked this pull request as ready for review May 11, 2021 21:59
@luchonetvv
Copy link
Member

@jselvamadrigal favor revisar porque no pasa la verificación de estado tus cambios antes de hacer merge en el branch develop.

Link del error: https://github.com/jugnicaragua/PICO/pull/68/checks?check_run_id=2560601378#step:6:747

Adjunto imagen
pico-error-pr-68

@jselvamadrigal
Copy link
Collaborator Author

@luchonetvv el error es una resolución de dependencia, el Docker desde donde se está generando el compilado no logra descargar las dependencias de Jasper report.

@luchonetvv
Copy link
Member

luchonetvv commented May 18, 2021

@jselvamadrigal y estamos usando reportes en PICO?

@luchonetvv luchonetvv requested a review from berroteran May 18, 2021 00:56
@jselvamadrigal
Copy link
Collaborator Author

@jselvamadrigal y estamos usando reportes en PICO?

Aparentemente sí, @luchonetvv deseas que en este pr realice una revisión de este problema de dependencia, cabe mencionar que localmente no se produce dicho error, lo que también quiera decir que el problema sea en la configuración del inspector utilizado.

@luchonetvv
Copy link
Member

@jselvamadrigal y estamos usando reportes en PICO?

Aparentemente sí, @luchonetvv deseas que en este pr realice una revisión de este problema de dependencia, cabe mencionar que localmente no se produce dicho error, lo que también quiera decir que el problema sea en la configuración del inspector utilizado.

@jselvamadrigal adelante revisa y nos propones la solución, de antemano gracias.

@jselvamadrigal jselvamadrigal changed the title Limpiando clase principal Implementando estructura de menú dinámicos May 18, 2021
@luchonetvv
Copy link
Member

luchonetvv commented May 18, 2021

Hola @jselvamadrigal revise el commit Actualizando dependencia de jasperreport pero tengo cierta incertidumbre del futuro de la dependencia de iText aunque sea soportado en un repositorio público de JasperReports y no Maven Central parece que solo se limitarán a hacerlo compatible más no mejorar o incorporar nuevas características sumado a que una de las razones por no utilizar la versión más reciente es por temas de Licencia veo que su futuro será estancado, intuyo que por eso no lo han publicado en Maven Central.

Otro dato desalentador es que tiene un problema de vulnerabilidad iText 2.1.7 XXE Vulnerability para tomar en cuenta.

Recomiendo evaluar este hilo de conversación del Issue del problema con la dependencia de iText JasperReports 6.14.0 fails at Maven stage: "Could not find artitact com.lowagie:itext:jar:2.1.7.js8".

Cómo última opción en ese hilo también recomiendan usar la bifurcación OpenPDF en dado caso hay inseguridad con respecto a iText, para más información de la bifurcación puedes visitar el proyecto siguiendo este enlace https://github.com/LibrePDF/OpenPDF

Nos dejas saber tu opinión con respecto a esta información y ver cuál será la mejor solución, de antemano muchas gracias.

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

Successfully merging this pull request may close these issues.

2 participants