search

creagia / laravel-redsys

Integrate your Laravel application with Redsys, the lead payment gateway in Spain.
MIT License
11 stars 3 forks source link

cURL error 28 al invocar la ruta de DS_MERCHANT_MERCHANTURL mediante Http::post #35

Closed gonbooster closed 5 months ago

gonbooster commented 11 months ago

Buenas noches,

Después de probar el entorno en modo local, me encuentro con un error cURL 28 de operación agotada después de 10002 milisegundos y 0 bytes recibidos. Puedes ver más detalles sobre este error en https://curl.haxx.se/libcurl/c/libcurl-errors.html.

A pesar de este error, la función __invoke de RedsysNotificationController se ejecuta correctamente. Como solución temporal, he implementado un bloque try-catch en las funciones Http::withoutVerifying() y Http::post. Aunque el error persiste, al menos permite que el sistema continúe funcionando con normalidad.

¿Es posible que este problema no sea un error per se, sino que podría requerir alguna configuración específica de mi Laravel?

Cuando el sistema esté en modo de prueba o producción, este problema no debería ocurrir, ya que el propio backend de Redsys invocará la URL almacenada en DS_MERCHANT_MERCHANTURL ¿Es correcto?

dtorras commented 11 months ago

Estás usando Laravel Sail en tu entorno local? En caso afirmativo, el problema es de Sail como explico en este otro issue https://github.com/creagia/laravel-redsys/issues/29#issuecomment-1744566624

Cuando tenga tiempo voy a volver a revisar si hay novedades en este tema o podemos cambiar la implementación para poder funcionar con Laravel Sail

gonbooster commented 11 months ago

¡Hola David!

Gracias!

Cómo es únicamente para probar el funcionamiento del merchantUrl y el testeo de la lógica del RedsysNotificationController no es crucial.

Aún así, podría mejorar agregando un bloque try-catch en las partes señaladas. De esta manera, podríamos utilizarlo sin problemas. La única desventaja sería que la solicitud se quedaría bloqueada por un tiempo hasta que ocurra el timeout.

dtorras commented 5 months ago

Hi, I'm closing this issue because this has been fixed in the latest release. If you need this in v2, I would appreciate a PR porting this solution to the v2 branch. Thanks.