Race condition in onNotify

Created on 14 January 2022, almost 3 years ago
Updated 14 March 2023, over 1 year ago

Problem/Motivation

I am experiencing the issue described in 🐛 The changes made to the order on the onNotify method are not applied on the onReturn method Fixed . I think that we have a race condition here.

Steps to reproduce

Well, this is kind of hard...

Proposed resolution

As this comment 🐛 The changes made to the order on the onNotify method are not applied on the onReturn method Fixed mentions: We should be doing something similar like this patch for commerce_saferpay: Implement order locking to prevent race conditions with async payment notification. Fixed

Remaining tasks

TBD

User interface changes

None

API changes

TBD

Data model changes

None

🐛 Bug report
Status

Fixed

Version

1.0

Component

Code

Created by

🇨🇭Switzerland Lukas von Blarer

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024