- Issue created by @Berdir
- Status changed to Needs review
about 1 year ago 1:50pm 2 August 2023 - 🇨🇭Switzerland Berdir Switzerland
First patch, again, reminder that this is work in progress.
- Assigned to ruvus
- 🇨🇭Switzerland Berdir Switzerland
Reroll.
The sleep issue is moved to 🐛 Hardcoded sleep on webhook callback blocks processes Needs review . The patch there conflicts with this though.
This is working quite well for us, but note that it changes PaymentSdkInterface, so technically has BC breaking changes.
- 🇨🇭Switzerland Berdir Switzerland
This is now finally in production for our client and so far is working well.
It's backwards compatible as the path stays the same, the gate query parameter just doesn't matter anymore.
I saw there's an option to use a sign, which might be a good idea, this doesn't use any data from the request but you could still at least get the system to do many possibly slow API requests.
- Merge request !9Resolve #3378753 Refactor webhooks to remove need for gate configuration and add event → (Open) created by Berdir
- 🇨🇭Switzerland Berdir Switzerland
Switched to a merge request and fixed a wrong route name there.