search

giansalex / peru-consult

Consulta libre de RUC y DNI
https://giansalex.github.io/peru-consult
MIT License
110 stars 65 forks source link

feat: add guzzle client #54

Closed MrJmpl3 closed 1 year ago

MrJmpl3 commented 2 years ago

I have some ideas to fix the code and tests also reformat the code, but this can be a major update and wish a fast update, I created a Guzzle Client

Note: Guzzle throw a exception when the URL Consult answer with a bad request or error, this can be used to check when the URL Consult is down.

Tengo algunas ideas para arreglar el codigo y los tests, ademas de reformatear el codigo, pero esto puede ser como una actualizacion mayor, y como queria una actualizacion rapida, solamente he creado un cliente Guzzle.

Nota: Guzzle lanza una excepcion cuando la URL de consulta responde con un error, esto podria ser usado para verificar si la URL de consulta esta caido.

sonarcloud[bot] commented 2 years ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

giansalex commented 2 years ago

Hola @MrJmpl3 , gracias, son bienvenidas las ideas que propones. Voy a corregir los tests data.

MrJmpl3 commented 2 years ago

@giansalex En los tests podrias usar mock responses para simular respuestas y asi no depende de los servidores de la SUNAT al momento de hacer tests.

giansalex commented 2 years ago

@MrJmpl3 si, se usa, se almacena en este repo. Pero por un tema de permisos de github-action no esta usando la url configurada. He actualizado master, puedes actualizar tu fork y rama.