[16.0][FIX] payment_redsys: Error processing redsys response #3309
Conversation
- /payment/redsys/result/<page> controller now process Redsys response and updates related transaction. Steps to reproduce: - Install payment_redsys módule - Configure payment method with test values from https://pagosonline.redsys.es/entornosPruebas.html as "Test mode" and publish. - Login into portal with "portal" user - Buy something and go to checkout screen, pay with Redsys. - Use an Autenticación Frictionless card numbres (4918019160034602,Expiration:12/34, CVV:123) - The payment will end with success message. - Press "Continuar" and you'll won't see the "Success message", instead you'll see a pending confirmation message. The endpoint that retrieves Redsys response doesn't process it and redirects the browser and losts the response information. All other módules call _handle_notification_data before the redirection.
|
@OCA/local-spain-maintainers |
|
ping @sergio-teruel |
|
Hola. ¿Pero tienes bien configurado el dominio para que Redsys te pueda enviar el POST con la confirmación? |
|
@sergio-teruel Si, estuve realizando las pruebas en varios entornos con el servidor correctamente configurado. Te dejo también la prueba en runboat con el modo de desarrollo y la configuración de https://pagosonline.redsys.es/entornosPruebas.html Recibía la respuesta de Redsys sin correctamente, pero redsys_result no procesaba el resultado de la transacción. redsys_return si procesa la respuesta con _handle_notification_data y ya traslada el estado de la transacción sin problemas. |
|
Hola @sergio-teruel, ¿se podría empujar para mergear? |
|
Pero veo que no es necesario... |
|
Buenos dias, Después de varias pruebas con el entorno de pruebas de Redsys, esto es totalmente necesario ya que cuando acabas de pagar en la web de redsys y de las al botón de continuar, redsys envia la respuesta a la url "/payment/redsys/result/redsys_result_ok?", entonces hace el comportamiento descrito en este PR, parece que el entorno de pruebas de redsys esta en modo síncrono asi que parece necesaria esta mejora. |
|
Hola @AntoniRomera |
Hola @sergio-teruel, como antes he dicho dentro de la configuración del TPV virtual se puede configurar de manera "síncrono/asíncrono", el comportamiento que tu describes se hace si esta de manera "asíncrono", el comportamiento que yo describo se hace cuando lo tienes de manera "síncrono", repito que acabo de realizar las pruebas con el TPV de test dado por redsys y sin este trozo de código modificado no funciona. Las pruebas realizadas, es ir a la web e intentar pagar el carrito. |
|
Ya.. si.. pero tenerlo así configurado es muy muy peligroso... ¿Que pasa si el usuario no pulsa continuar en la pasarela?... |
|
Creo que entonces lo mejor será poner en el README del módulo que se debería configurar del otro modo, ¿no? |
|
Aunque se ponga en el readme, la opción de configurarlo así y que no
funcione seguirá estando.
La corrección igualmente debería de estar ¿no? Junto con el aviso.
Saludos.
El mié, 5 jun 2024 19:44, Pedro M. Baeza ***@***.***>
escribió:
… Creo que entonces lo mejor será poner en el README del módulo que se
debería configurar del otro modo, ¿no?
—
Reply to this email directly, view it on GitHub
<#3309 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAFOA5OM2AN5MOFXATCY5ULZF5E77AVCNFSM6AAAAAA7K5SCMSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJQGYYTMNBZGE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
|
Promover una práctica peligrosa creo que no es lo más adecuado. ¿Qué razón hay para colocar esa configuración? |
|
Buenas tardes a todos, Revisando el problema que originaba este PR, sí que se recibe la respuesta en el endpoint que comentábais ( Por tanto, se podría descartar el PR para evitar confusiones. No obstante, lo que no se produce es la redirección automática de Redsys a Gracias, un saludo. |
|
Yo considero que es como funciona el sistema. Si no le das a continuar, no continúa... ¿no? |
|
Pensábamos que existiría el típico mensaje: Será redirigido en 5 segundos. Si no se le redirige automáticamente, haga click aquí. Si no existe esa posibilidad nada, no le damos más vueltas. Gracias! |
|
Ya, parecería lo lógico,pero aquí creo que depende de la propia Redsys, no de Odoo. Cierro entonces esto. Si consideráis que se deba hacer la apreciación en el README sobre ese modo asíncrono que se comentaba, adelante con el PR. @papulo79 ¿estás de acuerdo en cerrar esto? ¿Cuáles son las razones para utilizar eso otro modo que comentáis? |
|
Tras leer todo, parece más un problema de falta de información que de modo de funcionamiento. Por mi ok, cerremos el tema ya. |
Steps to reproduce:
The endpoint that retrieves Redsys response doesn't process it and redirects the browser and losts the response information.
All other payment modules call _handle_notification_data before the redirection.