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

Migración de request a urllib #20

Merged
merged 11 commits into from
Feb 17, 2021
Merged

Migración de request a urllib #20

merged 11 commits into from
Feb 17, 2021

Conversation

bjardon
Copy link
Contributor

@bjardon bjardon commented Feb 4, 2021

La biblioteca request está obsoleta y se recomienda migrar a alguna alternativa más segura. urllib ofrece la misma funcionalidad, también manejando las peticiones con el patrón de callbacks.

Reemplacé las llamadas a request con llamadas de la biblioteca de urllib, preservando la estructura de los métodos que ya existe. Reescribí la función sendRequest con estos cambios.

Los tests tenían algunos problemas en los datos que se enviaban al servidor, como las fechas de vencimiento de las tarjetas de prueba y los webhooks. Incluí una nueva variable que calcula un año válido según el año actual, para evitar fallas futuras. En el caso del test de webhooks, abrí un RequestBin nuevo en la url de prueba, y agregué comentarios para futuros contributors.

Todos los tests en local pasaron; pero si algún core maintainer puede revisar el código, lo agradecería.

@bjardon
Copy link
Contributor Author

bjardon commented Feb 4, 2021

Este PR resuelve #17 y #7
El resultado de npm audit es

                       === npm audit security report ===                        
                                                                                
found 0 vulnerabilities
 in 212 scanned packages

@carloshe-perez carloshe-perez merged commit 858899e into open-pay:master Feb 17, 2021
@jemy-dasm-openpay jemy-dasm-openpay mentioned this pull request Jun 9, 2021
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