That is, we're sending a bad signature. Figure out what we're doing wrong and fix it.
30 so far seen today.
That is, we're sending a bad signature. Figure out what we're doing wrong and fix it.
30 so far seen today.
I can trigger it if I put a plus sign in an street address or city. Maybe we can filter those out completely?
If we're removing the address fields from the dlocal form completely (see T308464) that will also fix it, but I feel like we should add the + sign scrubbing as well in case they are used elsewhere.
Change 800033 had a related patch set uploaded (by Ejegg; author: Ejegg):
[mediawiki/extensions/DonationInterface@master] Filter out + signs from DLocal address fields
Still seeing the invalid control error and there is no address from request field that can screwed up the signature.{"id":"120123033","ts":"20220610121743","referrer":"en.wikipedia.org\/","utm_source":"B2122_0527_enZA_dsk_p1_lg_txt_109.no-LP.cc","utm_medium":"sitenotice","utm_campaign":"C2122_enZA_dsk_FR","utm_key":"otherAmt_1~ptf_1~vw_1920~vh_969~time_22","language":"en","country":"ZA","form_amount":"ZAR 1248.00","payments_form":"astropay.JimmyQuote","source_name":"DonationInterface","source_type":"payments","source_host":"payments1006","source_run_id":3966,"source_version":"8c6208c25c1f66ee67bfc7ef975b4e5b9051aebb","source_enqueued_time":1654863463}
I'm not finding that one specifically, but I do still see some instances of that error, all from donors who use a + sign in their email address. We can collect their full email address for ourselves and just send DLocal a version with everything from the + sign to the @ sign stripped out.
Change 808986 had a related patch set uploaded (by AndyRussG; author: AndyRussG):
[mediawiki/extensions/DonationInterface@master] Remove removal of "+" symbols in AstroPay signature
Change 800033 abandoned by Ejegg:
[mediawiki/extensions/DonationInterface@master] Filter out + signs from DLocal address fields
Reason:
AndyRussG found a simpler way to fix it
Change 808986 merged by jenkins-bot:
[mediawiki/extensions/DonationInterface@master] Remove removal of "+" symbols in AstroPay signature
Oh hey, this went out on Jun 29th. Seems to have solved the problem. Thanks, @AndyRussG!
$ zgrep -ci 'invalid control' payments-astropay-202206[23]*
payments-astropay-20220620.gz:35
payments-astropay-20220621.gz:30
payments-astropay-20220622.gz:35
payments-astropay-20220623.gz:105
payments-astropay-20220624.gz:70
payments-astropay-20220625.gz:45
payments-astropay-20220626.gz:30
payments-astropay-20220627.gz:20
payments-astropay-20220628.gz:5
payments-astropay-20220629.gz:40
payments-astropay-20220630.gz:0
$ zgrep -ci 'invalid control' payments-astropay-202207*
payments-astropay-20220701.gz:0
payments-astropay-20220702.gz:0
payments-astropay-20220703.gz:0
payments-astropay-20220704.gz:0
payments-astropay-20220705.gz:0
payments-astropay-20220706.gz:0
payments-astropay-20220707.gz:0