GitLab

changed the description

@OpenPymeMx revisando el código, me encontré con que no se estaban considerando las excepciones a la hora de comunicarse con los webservices de finkok, se usa la libreria suds que tiene bien definidas las excepciones que lanza al intentar comunicarse con los webservice!

added 2h of time spent at 2018-09-24

added Bug S labels

¿estas excepciones son las el timbrado de facturas verdad?

@majose.cruz si, son excepciones que se arrojan si falla de alguna manera el uso de los web services de finkok!

resolved all discussions

added 1 commit

7c797c26 - fix(params.pac): se cachan las excepciones de suds

Compare with previous version

@OpenPymeMx los demás que opinan? por un lado tenemos que no se estaban considerando las excepciones que se podían presentar al comunicarse con el webservice de finkok, de ahí salio este MR, y por otro lado tenemos que ya queremos migrar la librería suds que ya tiene mas de 3 años sin mantenimiento, por la de requests! creo que mientras no se haga el cambio de la librería, entonces hacer robusto el código cachando estas excepciones es bueno, ya que cuando se haga el cambio de librería sabremos que hay que considerar las excepciones que se puedan generar con la comunicación con el webservice!