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

Payment from expired session stored as completed #15230

Closed
pierre-dedi opened this issue Aug 16, 2023 · 2 comments
Closed

Payment from expired session stored as completed #15230

pierre-dedi opened this issue Aug 16, 2023 · 2 comments
Labels
Potential Bug Potential bugs or bugfixes, that needs to be reproduced.

Comments

@pierre-dedi
Copy link

Sylius version affected: 1.11.7

Description
Some payments are recorded in the database with the status "completed" while the log returned by the bank contains the following error message: "The consumer is not redirected on payment web pages and session is expired". Thus, when a payment considered valid by the bank is made afterwards, it is not recorded by Sylius.
I have 2 questions :

  • Is it possible to add an "Expired" or "Failed" status for the failed payment in this case?
  • Is the fact that the second payment is not recorded linked to the fact that the first one is considered "completed" ?

Steps to reproduce
I haven't been able to reproduce the payment session expiration yet.

@jakubtobiasz
Copy link
Contributor

Hello @pierre-dedi 👋🏼!

Sylius by default comes with PayPal plugin only, so I can't understand your case fully. Do you develop your own plugin or use a 3rd one? If you're using a community-driven plugin the best choice is to open an issue in the plugin's repository.

@jakubtobiasz jakubtobiasz added Potential Bug Potential bugs or bugfixes, that needs to be reproduced. Help Wanted Issues needing help and clarification. and removed Help Wanted Issues needing help and clarification. labels Aug 29, 2023
@NoResponseMate
Copy link
Contributor

Hi @pierre-dedi 👋

Since there's not enough information to reproduce the issue as well as it being a bit stale I'm closing it. Please open another one if it's still a problem and you can provide more info.

Cheers 🍻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Potential Bug Potential bugs or bugfixes, that needs to be reproduced.
Projects
None yet
Development

No branches or pull requests

3 participants