| MBeat33 | |
| Oct 26 2022, 1:52 PM |
| F35715542: Screen Shot 2022-11-07 at 12.50.19 PM.png | |
| Nov 7 2022, 5:53 PM |
| F35715539: Screen Shot 2022-11-07 at 12.50.11 PM.png | |
| Nov 7 2022, 5:53 PM |
| F35692975: Screen Shot 2022-11-01 at 4.36.12 PM.png | |
| Nov 1 2022, 8:38 PM |
| F35691457: Screen Shot 2022-11-01 at 11.30.58 AM.png | |
| Nov 1 2022, 3:33 PM |
| F35630839: Forget Me2022-10-26 at 9.37.33 AM.png | |
| Oct 26 2022, 1:52 PM |
Yamini spotted and I was able to replicate that when choosing the Forget Me button, it stops and says:
Configuration key 'payment-provider/cc/class' does not exist
The last successful use was October 22nd, 2022 7:02 PM for activity: 1193250 Privacy Request, so the error must be pretty recent.
Here is what we see:
Maybe this is related to T318283: build Adyen API into Civi Forget Me button? The donors we tested were ones without Adyen contribs.
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Error handling for contribution forgetme | wikimedia/fundraising/crm | master | +25 -8 |
Oops, I think I know what caused this. Should be an easy fix. Sorry for not testing thoroughly enough!
Change 851176 had a related patch set uploaded (by Ejegg; author: Ejegg):
[wikimedia/fundraising/crm@master] Error handling for contribution forgetme
Change 851176 merged by jenkins-bot:
[wikimedia/fundraising/crm@master] Error handling for contribution forgetme
OK @MBeat33 the fix is deployed. It no longer throws the error when loading the initial screen on that contact. I haven't yet pressed the button to do the actual forgetting, but I think that should work too.
Thanks, @Ejegg I tested on a non-Adyen donor and it worked, but for an Adyen donor Civi responded with:
Oh shoot! There's one config file update I missed, and I think a bit of network configuration that we'll need. Sorry again.
OK, the config file is updated so the forgetme button will now make the GDPR requests against the live rather than test Adyen endpoint.
@Jgreen or @Dwisehaupt when you have time, could you please see about allowing outbound access from civi1001 to yet another Adyen API endpoint? It needs to be able to hit https://ca-live.adyen.com/ca/services/DataProtectionService/v1 . Thank you!
I have added the config for the ipset and enabled it. Also deployed the iptables config.
When testing, it looks like that url redirects to https://authn-live.adyen.com so looking at adding that in too.
Added in the config for ipset and iptables rules for adyen.authn-live and enabled it all. Verified the connection with wget and it is successful.
Awesome, thank you both. I just tested with an Adyen and a non-Adyen donor, and it looks great.
Not sure if this should be a spinoff Task, but I got the following error when forgetting cid=13211031
https://civicrm.wikimedia.org/civicrm/a/#/forgetme/forget/13211031
This cid is one of the older deletion request for which I was re-applying Forget Me, in order to delete the Adyen data.
Looks like there is a trailing comma in the list there and thus it's seeing a non-int value. @Ejegg could you take a look when you get a chance.
