search

open-pay / openpay-node

Openpay Node.js bindings
29 stars 16 forks source link

Migración de request a urllib #20

Closed bjardon closed 3 years ago

bjardon commented 3 years ago

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 commented 3 years ago

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

                       === npm audit security report ===                        

found 0 vulnerabilities
 in 212 scanned packages