SEPE plugin: xmlseclibs.php has newer versions

[ywarnier]

A @nosolored (no urgente)

La librería xmlseclibs usada en el plugin SEPE es bastante antigua. Siendo una de seguridad, creo que valdría la pena analizar si es que se puede usar la última versión, y probablemente que en este caso se pueda usar a través de composer?

Ver https://github.com/robrichards/xmlseclibs y https://packagist.org/packages/robrichards/xmlseclibs

[nosolored]

Hacemos pruebas al respecto y realizaremos un pull request cuando esté lista. No parece que implique mucha dificultad la integración en composer.

[jmontoyaa]

@nosolored ping

[nosolored]

https://github.com/chamilo/chamilo-lms/pull/2197

[nosolored]

Julio los cambios realizados con la librería no muestra errores, pero en el proceso de validación con la herramienta del validador no funciona correctamente. Creo que falla en el /plugin/sepe/ws/service.php en $soap->handle($xml); Te adjunto el validador test.zip

[jmontoyaa]

Tratar de reproducir el error en https://11.chamilo.org/

[nosolored]

Para probar en https://11.chamilo.org/ hace falta cambiar la linea 910 del /plugin/sepe/ws/ProveedorCentroTFWS.wsdl para indicar el dominio <soap:address location="http://personal-domain.com/plugin/sepe/ws/service.php"/> a <soap:address location="https://11.chamilo.org/plugin/sepe/ws/service.php"/>

[nosolored]

Hola Julio, ¿has podido cambiar el fichero ProveedorCentroTFWS.wsdl para realizar las pruebas en 11.chamilo.org?

[jmontoyaa]

Eso esta mal ... En realidad el plugin debería de dar un archivo por defecto, presente en el repositorio:

ProveedorCentroTFWS.dist.wsdl

Luego, se tiene que invitar al usuario ha crear un nuevo archivo y modificarlo:

ProveedorCentroTFWS.wsdl

Algo como el mecanismo que usamos con configuration.dist.php/configuration.php

[ywarnier]

De acuerdo con @jmontoyaa

[ywarnier]

Supongo que @nosolored no podrá verlo pronto, así que postergo para la 1.11.8

[nosolored]

Hola, Es correcto el usar el archivo ProveedorCentroTFWS.dist.wsdl, ningun inconveniente con esto @ywarnier, nosotros no podemos probar nada en https://11.chamilo.org si no cambiais el archivo, no tenemos acceso a él. En nuestras demos como ya comentamos el validador no funciona bien, algo falla.

[jmontoyaa]

Acabo de agregar ese cambio de ".dist" también he creado el archivo en 11.chamilo.org:

https://11.chamilo.org/plugin/sepe/ws/ProveedorCentroTFWS.wsdl

[NicoDucou]

@nosolored El archivo esta listo en 11.chamilo.org para que prueben y indicqn como reproducir el problema.

[nosolored]

ProvCentTFVal.zip Aquí está el validador que proporciona la sede de la administración del SEPE para verificar si funciona correctamente. Tan sólo hay que editar el fichero run.cmd y cambiar el path en dos lineas por las correspondientes en el directorio Windows que se ubique. A continuación ejecutarlo y muestra los errores que encuentra, debería salir todo "OK" para pasar la prueba.

[NicoDucou]

No tengo un windows para probar, salen errores cuando lo probas con https://11.chamilo.org/plugin/sepe/ws/ProveedorCentroTFWS.wsdl ?

[nosolored]

Disculpad que no lo hayamos indicado, lo hemos probado y falla, por eso os explicábamos cómo probarlo con el validador El problema de las aplicaciones que tienen relación con la Administración Pública es que son muy "delicadas", si cambias la versión de alguna librería es muy fácil que dejen de funcionar, ocurre con frecuencia. Si queréis probarlo vosotros, desde aquí podéis descargar una máquina virtual de Windows que se puede montar tanto en Linux como MAC: https://developer.microsoft.com/en-us/microsoft-edge/tools/vms/ El SEPE solo ha hecho el validador para Windows, no hay otra forma.

[stale[bot]]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.