search

PabloCastellano / bormeparser

A Python library for parsing BORME files (Boletín Oficial del Registro Mercantil in Spain).
GNU General Public License v3.0
47 stars 20 forks source link

Path may not exist & connection issues in download #12

Closed j6e closed 6 years ago

j6e commented 7 years ago

Changes to be committed: modified: bormeparser/borme.py modified: bormeparser/download.py

borme.py: save_to_file -Comprobacion de que la carpeta donde se quiere escribir existe. De no existir se crea.

download.py: download_url -Anyadido el argumento keyword 'try_again' con valor default 0. De esta forma si se produce un error en el request.get (perdida de paquetes, perdida de conexion, etc) se reintenta de forma recursiva hasta 3 veces. -Anyadido log de warnings para dejar constancia de que algo malo ha ocurirido en la descarga. -En caso de fallar 3 veces se lanza una excepcion. Seguramente hayamos perdido la conexion o el servidor sea inalcanzable.

Esto es para resolver los issues https://github.com/PabloCastellano/libreborme/issues/49 y https://github.com/PabloCastellano/libreborme/issues/48

coveralls commented 7 years ago

Coverage Status

Coverage decreased (-0.2%) to 79.976% when pulling cafc0b1d113ff7ae4a19c347544592de2541d444 on j6e:master into 7cd468e20b188326849402062e272355e114e6ce on PabloCastellano:master.