[01:59:14] (03CR) 10AndyRussG: [C: 03+2] "Looks great, thanks so much for this!" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/902827 (https://phabricator.wikimedia.org/T333014) (owner: 10Wfan) [01:59:50] (03Merged) 10jenkins-bot: Add subscription_max_amount to prevent higher than init payment's recurring charge [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/902827 (https://phabricator.wikimedia.org/T333014) (owner: 10Wfan) [02:14:06] (03CR) 10AndyRussG: "In view of @ejegg's explanation (see inline comments) this change does seem fine! (Just need to update the commit message.) Thanks so much" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/900717 (https://phabricator.wikimedia.org/T324297) (owner: 10Wfan) [02:29:06] 10Fundraising-Backlog, 10Diffusion-Repository-Administrators, 10Projects-Cleanup, 10Wikimedia-GitHub: Archive the FundraisingEmailUnsubscribe extension - https://phabricator.wikimedia.org/T224764 (10Pppery) [06:00:46] (03CR) 10CI reject: [V: 04-1] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] (REL1_35) - 10https://gerrit.wikimedia.org/r/902939 (owner: 10L10n-bot) [06:41:09] (03CR) 10AndyRussG: [C: 03+2] "Cool!!!!!" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/902177 (https://phabricator.wikimedia.org/T324302) (owner: 10Ejegg) [06:42:54] (03Merged) 10jenkins-bot: Respect result.redirect in dlocal.js [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/902177 (https://phabricator.wikimedia.org/T324302) (owner: 10Ejegg) [06:51:05] (03CR) 10AndyRussG: "Thanks!! I'm getting the same result as @jgleeson, with only the $wgDonationInterface3DSRules setting mentioned." [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/902204 (https://phabricator.wikimedia.org/T332831) (owner: 10Ejegg) [07:48:39] 10fundraising-tech-ops, 10Infrastructure-Foundations, 10SRE, 10netops: Please update bootp helper on pfw3-eqiad to point to frpm1002 for fundraising subnets - https://phabricator.wikimedia.org/T332939 (10ayounsi) 05Open→03Resolved a:03ayounsi Done! [09:04:01] 10Fundraising-Backlog, 10Data-Engineering, 10Event-Platform Value Stream, 10MediaWiki-extensions-EventLogging, and 2 others: Determine which remaining legacy EventLogging schemas need to be migrated or decommissioned - https://phabricator.wikimedia.org/T282131 (10phuedx) [10:56:41] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10fundraising-tech-ops, 10FR-dlocal: Add smashpig listener server config to allow Authorization header - https://phabricator.wikimedia.org/T332891 (10jgleeson) a:03jgleeson [11:04:29] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: Add multi-byte support when checking max field lengths in CiviCRM. - https://phabricator.wikimedia.org/T332991 (10jgleeson) thanks for the review @Eileenmcnaughton :) [11:34:24] (03PS2) 10Jgleeson: Refactor how we get HTTP headers in the dLocal Listener [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/902405 (https://phabricator.wikimedia.org/T332637) [11:36:02] (03PS3) 10Jgleeson: Refactor how we get HTTP headers in the dLocal Listener [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/902405 (https://phabricator.wikimedia.org/T332637) [11:44:39] (03PS4) 10Jgleeson: WIP: add rejected IPN handling to dlocal listener [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/901701 (https://phabricator.wikimedia.org/T332637) [11:45:08] (03CR) 10CI reject: [V: 04-1] WIP: add rejected IPN handling to dlocal listener [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/901701 (https://phabricator.wikimedia.org/T332637) (owner: 10Jgleeson) [12:39:45] (03CR) 10Jgleeson: [C: 04-1] "I'm testing this now, and I do not think dLocal is sending us different casing across Sandbox and Production. I also don't think we need t" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/902765 (owner: 10Jgleeson) [12:54:59] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10fundraising-tech-ops, 10FR-dlocal: Add smashpig listener server config to allow Authorization header - https://phabricator.wikimedia.org/T332891 (10jgleeson) a:05jgleeson→03None [13:14:04] (03CR) 10Jgleeson: "thanks for the review!" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/902405 (https://phabricator.wikimedia.org/T332637) (owner: 10Jgleeson) [13:25:47] fr-tech it looks like AndyRussG +2ed Wfan's patch adding 'subscription_max_amount' to the UPI payment request so I'm gonna deploy this now in time for the dlocal test call today [13:41:24] gotta pop out to pick the kids up. if anyone else feels like deploying payments with smashpig to get ^^^ out go for it! if not I'll do it when I get back. [14:30:09] ok I'll carry on with that deploy [15:00:47] ok cool so this should be pretty straight forward [15:00:55] we've got a new tag for the smashpig update [15:00:59] https://packagist.org/packages/wikimedia/smash-pig [15:01:11] so I'll update DI to bump up the smashpig version now [15:02:04] (03PS1) 10Jgleeson: update smash-pig to 0.8.4.7 [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/903271 [15:02:33] (03CR) 10Jgleeson: [C: 03+2] update smash-pig to 0.8.4.7 [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/903271 (owner: 10Jgleeson) [15:03:02] so now we need to pull that into payments and payments vendor [15:04:45] (03Merged) 10jenkins-bot: update smash-pig to 0.8.4.7 [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/903271 (owner: 10Jgleeson) [15:09:49] (03PS1) 10Jgleeson: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/903272 [15:14:01] hi fr-tech! [15:14:07] hey ejegg [15:14:08] (03PS1) 10Jgleeson: Update DonatonInterface, vendor and Smashpig version [core] (fundraising/REL1_39) - 10https://gerrit.wikimedia.org/r/903273 [15:14:13] hi ejegg [15:14:27] heyyy [15:14:36] so what can I help with for dlocal? Is someone already doing the return processing for 3ds? [15:14:54] ejegg: there are some comments on your patch there [15:15:14] thanks! I'll look at those first [15:15:46] ejegg: basically it's just that I got the same issue as Jack: a fleeting interstitial visa 3d-ish screen and then a fail page [15:16:16] with the "Missing required parameters in request" in the log [15:16:35] I didn't get as afar as actually understanding what's going on and what more is needed, but yeah it seems like an urgent one [15:16:41] ok, that means the first patch is doing its job [15:17:03] ejegg: ok if that's the desired effect then I can +2 that [15:17:04] And we just need to write the second patch for the return processing [15:17:22] Yeah, previously we weren't even redirecting to the 3ds page [15:17:30] right [15:17:44] ah also I didn't get what the test class change would do [15:18:05] Oh hmm, i forget. maybe that was unintended? [15:18:14] hmmm [15:18:43] ah no, that was intended to set a controlled, very specific setting for 3DSRules [15:18:45] ejegg:FYI there are few dlocal threads. the 3ds stuff is one, I don't think anyone started on a patch for the suggested redirect handling. The other is the UPI field 'subscription_max_amount' https://phabricator.wikimedia.org/T333014 which is getting deployed now so Rahki can test and the last is the ongoing dance with the Authorization/authorization/HTTP_AUTORIZATION header [15:18:47] https://phabricator.wikimedia.org/T332951 [15:19:11] since the gateway-specific DlocalGateway3DSRules will override any DonationInterface3DSRules [15:19:27] I wanted to make sure the tests would override anything people had locally when running the tests [15:19:29] ejegg: right but it doesn't presently have any impact on tests? [15:19:39] oh ok [15:19:50] some tests were failing till I took it out [15:19:55] perhaps just locally [15:20:30] ok sounds good, maybe just add a comment to that effect right there, and I'll +2? [15:20:34] (03CR) 10Ejegg: Dlocal: set use_3d_secure when configured (031 comment) [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/902204 (https://phabricator.wikimedia.org/T332831) (owner: 10Ejegg) [15:20:42] ah, a code comment? [15:20:44] ok, can do [15:20:48] yee thx! [15:22:08] (03CR) 10Damilare Adedoyin: [C: 03+2] Log rawResponse on recurring charge failure [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/885902 (https://phabricator.wikimedia.org/T239770) (owner: 10Ejegg) [15:22:45] (03CR) 10Jgleeson: [C: 03+2] Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/903272 (owner: 10Jgleeson) [15:26:40] (03PS5) 10Ejegg: Dlocal: set use_3d_secure when configured [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/902204 (https://phabricator.wikimedia.org/T332831) [15:27:13] ok, added the comment ^^^ [15:27:18] (and reformatted the file) [15:27:29] I'm curious why we're not linting that one [15:28:28] ah it was already all tabs - maybe it doesn't check for that specific indenting [15:28:42] ejegg: https://gerrit.wikimedia.org/r/c/mediawiki/extensions/DonationInterface/+/902204/4/dlocal_gateway/Dlocal3DSecure.php#6 [15:28:52] stray comment [15:29:02] oh yep, copypasta [15:29:05] will scrub [15:29:18] ah nice catch jgleeson [15:29:30] also jgleeson since you'd been reviewing that too, I should wait to +2 [15:29:42] (03PS6) 10Ejegg: Dlocal: set use_3d_secure when configured [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/902204 (https://phabricator.wikimedia.org/T332831) [15:29:45] it's fine AndyRussG. happy to see it +2ed! [15:29:47] ty [15:29:52] thank you [15:29:53] jgleeson: ah great thanks much! [15:31:04] (03CR) 10AndyRussG: [C: 03+2] "Cool beans, thanks so much @ejegg and @jgleeson!!" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/902204 (https://phabricator.wikimedia.org/T332831) (owner: 10Ejegg) [15:33:10] 8 minutes and counting https://gerrit.wikimedia.org/r/c/mediawiki/extensions/DonationInterface/+/903272 [15:36:40] (03Merged) 10jenkins-bot: Log rawResponse on recurring charge failure [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/885902 (https://phabricator.wikimedia.org/T239770) (owner: 10Ejegg) [15:37:14] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Civi-Dedupe, 10FR-Email: Civi dedupe: automatically dedupe alternate family names - https://phabricator.wikimedia.org/T280696 (10SHust) This is it, thanks @Eileenmcnaughton! [15:37:22] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Email: Enable sending Civi contact hashes in Acoustic emails - https://phabricator.wikimedia.org/T333182 (10Ejegg) [15:38:05] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/903272 (owner: 10Jgleeson) [15:38:11] (03Merged) 10jenkins-bot: Dlocal: set use_3d_secure when configured [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/902204 (https://phabricator.wikimedia.org/T332831) (owner: 10Ejegg) [15:39:14] (03CR) 10Jgleeson: [C: 03+2] "let's get it out there!" [core] (fundraising/REL1_39) - 10https://gerrit.wikimedia.org/r/903273 (owner: 10Jgleeson) [15:41:29] 10Fundraising-Backlog, 10fr-donorservices: How can we enable upselling of existing recurring donation amount? - https://phabricator.wikimedia.org/T325317 (10Ejegg) This work would probably be needed for us to enable this functionality: https://phabricator.wikimedia.org/T333182 > * What fields need to be added... [15:47:24] (03Merged) 10jenkins-bot: Update DonatonInterface, vendor and Smashpig version [core] (fundraising/REL1_39) - 10https://gerrit.wikimedia.org/r/903273 (owner: 10Jgleeson) [15:50:19] yay [15:54:44] 10Fundraising-Backlog, 10fundraising-tech-ops, 10Epic: [FR Analytics]: Overhaul /srv/br - https://phabricator.wikimedia.org/T178953 (10Jgreen) [15:54:54] 10Fundraising-Backlog, 10fundraising-tech-ops: [FR Analytics]: move frdev1001:/srv/br/analytics out to it's own directory - https://phabricator.wikimedia.org/T258601 (10Jgreen) 05duplicate→03Resolved [15:56:39] Thanks Andy :) [15:59:30] (03PS5) 10Wfan: Move PlaceholderFiscalNumber to dlocal_gateway [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/900717 (https://phabricator.wikimedia.org/T324297) [16:00:12] (03PS1) 10Ejegg: Dlocal: Handle return from 3ds authentication [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/903294 (https://phabricator.wikimedia.org/T332831) [16:01:07] thank u wfan :) [16:06:44] 10Wikimedia-Fundraising-Banners, 10Wikipedia-iOS-App-Backlog, 10iOS Release FY2022-23: Japan fundraising messages in iOS app - https://phabricator.wikimedia.org/T332229 (10Seddon) [16:19:28] 10Fundraising Sprint Fish HEAD^, 10Fundraising-Backlog: Add new dLocal UPI field 'subscription_max_amount' - https://phabricator.wikimedia.org/T333014 (10jgleeson) We just deployed this and it looks like it fixes the issue! thanks all {F36929753, size=full} [16:25:32] !log payments-wiki upgraded from 36366f64 to f5e262d1 [16:25:36] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [16:34:08] 10Wikimedia-Fundraising-Banners, 10Wikipedia-Android-App-Backlog (Android Release FY2022-23): Japan fundraising messages in Android app - https://phabricator.wikimedia.org/T332230 (10ABorbaWMF) [17:29:48] 10Fundraising-Backlog: One-time UPI Payment in India - QR code not reading - https://phabricator.wikimedia.org/T333205 (10RKumar_WMF) [17:31:32] 10Fundraising-Backlog: One-time UPI Payment in India - QR code not reading - https://phabricator.wikimedia.org/T333205 (10RKumar_WMF) {F36929814} [17:54:56] 10Fundraising-Backlog: One-time UPI Payment in India - QR code not reading - https://phabricator.wikimedia.org/T333205 (10RKumar_WMF) Closing since the issue in on dLocal's side. [17:55:12] 10Fundraising-Backlog: One-time UPI Payment in India - QR code not reading - https://phabricator.wikimedia.org/T333205 (10RKumar_WMF) 05Open→03Resolved a:03RKumar_WMF [18:04:53] 10Fundraising-Backlog, 10FR-Japan: Make the Japan "V1" payment wiki variant the new control - https://phabricator.wikimedia.org/T329807 (10TSkaff) Online team requests to make V1 control thank you! cc @spatton @AKanji-WMF [18:17:17] fr-tech dwisehaupt Jeff_Green to u-turn completely on what we thought on Friday. I'm now pretty sure we are stripping the Authorization header on the wmf side somewhere. The confusion around our code changing the casing made me wonder what happens if we sent the header over all lowercase as 'authorization'. Our code eventually expects it to be transformed to lowercase so I figured it should [18:17:19] still work and it does! However if I update the header to 'Authorization', I get the "header is missing" exception in the code. The code converts this header to lowercase in all cases so even if it was received as 'Authorization' it would be converted to 'authorization' and this exception wouldn't happen so the fact that it is happening makes me fairly confident we're stripping it at the server [18:17:21] level. [18:19:14] jgleeson: do we have a method of sending a request with the expected headers for testing? [18:19:43] yeah. I'm using Postman with a request I built from the stuff we've received in the past [18:20:07] I can demo on a call if you'd like and share the request [18:20:28] ok hold a sec, lemme look at the task and get oriented [18:24:13] jgleeson: can you fire off one of those so I can see what it looks like in the logs? [18:24:33] the successful one or the failing one [18:24:55] ha I wasn't prepared for there to be a successful option :-) [18:25:00] I guess the failing one [18:25:26] ok cool. if you want a grep anchor you can use jwales@example.com [18:25:37] sending now [18:25:42] ok [18:26:01] sent [18:26:26] I think I see it, not sure, PMing you the IP in question [18:37:35] (03PS2) 10Ejegg: Dlocal: Handle return from 3ds authentication [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/903294 (https://phabricator.wikimedia.org/T332831) [18:38:29] ok AndyRussG, ^^^ is handling the 3ds redirect [18:39:06] I've added 'BRL' => ['BR'] to my 3DSRules in localsettings-private [18:39:17] to force the redirect [18:46:01] ejegg: cool thx! yeah I've been testing with INR / IN [18:56:17] fr-tech I'm gonna deploy the latest on civi to get out https://phabricator.wikimedia.org/T332991 and get that donor their thank you email [18:57:21] (03PS1) 10Jgleeson: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/903320 [19:00:07] (03CR) 10Jgleeson: [C: 03+2] Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/903320 (owner: 10Jgleeson) [19:01:12] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/903320 (owner: 10Jgleeson) [19:03:03] fr-tech, the latest thinking on the missing IPN Authorization header is that we're possibly losing it when switching from https => http in the nginx=>apache hand off. [19:06:07] !log civicrm upgraded from 09373b9d to db3b727e [19:06:11] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [19:09:31] cstone: I think we can abandon this patch https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/SmashPig/+/902765 [19:09:47] sounds good jgleeson [19:11:02] (03Abandoned) 10Jgleeson: dLocal IPN fix: set the headers to lowercase [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/902765 (owner: 10Jgleeson) [19:28:14] (03CR) 10Ejegg: "Thanks for the move! Maybe undo the reformatting, though?" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/900717 (https://phabricator.wikimedia.org/T324297) (owner: 10Wfan) [19:30:55] (03PS2) 10Ejegg: Remove more watchdogs in wmf_civicrm [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/902217 (https://phabricator.wikimedia.org/T288585) (owner: 10Eileen) [19:30:59] (03CR) 10Ejegg: [C: 03+2] Remove more watchdogs in wmf_civicrm [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/902217 (https://phabricator.wikimedia.org/T288585) (owner: 10Eileen) [19:36:37] (03PS6) 10Wfan: Move PlaceholderFiscalNumber to dlocal_gateway [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/900717 (https://phabricator.wikimedia.org/T324297) [19:40:35] (03PS4) 10Wfan: Add upi id field for direct upi [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/902609 (https://phabricator.wikimedia.org/T324295) [19:46:30] (03CR) 10Eileen: Log rawResponse on recurring charge failure (031 comment) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/885902 (https://phabricator.wikimedia.org/T239770) (owner: 10Ejegg) [19:51:15] (03Merged) 10jenkins-bot: Remove more watchdogs in wmf_civicrm [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/902217 (https://phabricator.wikimedia.org/T288585) (owner: 10Eileen) [19:52:21] 10Fundraising-Backlog, 10FR-dlocal: Pending Transaction Resolver support for dLocal - https://phabricator.wikimedia.org/T333220 (10jgleeson) [19:57:40] 10Fundraising-Backlog: UZS Donation Form Error - https://phabricator.wikimedia.org/T332945 (10AKanji-WMF) @EMartin Can you please confirm with Worldline as to whether we support payments from UBZ? [20:00:46] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10fundraising-tech-ops, 10FR-dlocal: Add smashpig listener server config to allow Authorization header - https://phabricator.wikimedia.org/T332891 (10AKanji-WMF) 05Open→03Resolved [20:00:49] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10FR-dlocal, 10Patch-For-Review: Handle REJECTED IPNS for Dlocal - https://phabricator.wikimedia.org/T332637 (10AKanji-WMF) [20:07:28] 10Fundraising Sprint Fish HEAD^, 10Fundraising-Backlog: Testing: Paytm recurring remains in pending status - https://phabricator.wikimedia.org/T332835 (10AKanji-WMF) [20:11:22] 10Fundraising-Backlog: Testing: Issues with One time donation through UPI on new Dlocal API - https://phabricator.wikimedia.org/T332830 (10AKanji-WMF) 05Open→03Resolved [20:16:28] (03PS2) 10Eileen: Remove watchdog from failmail [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/902221 (https://phabricator.wikimedia.org/T288585) [20:18:52] 10Fundraising-Backlog: dLocal: add config and code to allow switching between UPI monthly and on-demand - https://phabricator.wikimedia.org/T332816 (10AKanji-WMF) 05Open→03Resolved [20:24:44] 10Fundraising-Backlog: Error after Civi import - https://phabricator.wikimedia.org/T332810 (10AKanji-WMF) 05Open→03Resolved [20:28:36] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: Paypal EC Pending Transaction Resolver failed (weekend failure) - https://phabricator.wikimedia.org/T332599 (10AKanji-WMF) [20:28:38] 10Fundraising-Backlog: Delete pending messages in the recurring queue consumer - https://phabricator.wikimedia.org/T332690 (10AKanji-WMF) 05Open→03Declined [20:36:47] (03PS2) 10Eileen: Remove more watchdogs from civicrm_module [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/902218 (https://phabricator.wikimedia.org/T288585) [20:36:54] 10Fundraising-Backlog: php_network_getaddresses: getaddrinfo failed: Temporary failure in name resolution - https://phabricator.wikimedia.org/T318956 (10AKanji-WMF) 05Open→03Resolved [20:37:30] (03CR) 10Eileen: "Hey wfan - thanks for looking." [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/902218 (https://phabricator.wikimedia.org/T288585) (owner: 10Eileen) [20:39:19] 10Fundraising-Backlog: Declines in Italy for no 3DS - https://phabricator.wikimedia.org/T332950 (10EMartin) 3DS has had a favorable impact: FinecoBank went from 0% to 93%. Credito Emeliano went from 13% to 93%. The only one worth monitoring is Credit Agricola, which has stayed at 11% approval rate [20:41:32] 10Fundraising-Backlog: Intermittent Apple Pay timeouts when testing locally - https://phabricator.wikimedia.org/T318808 (10AKanji-WMF) 05Open→03Declined [20:44:18] 10Fundraising-Backlog: AmazonPay script errors when testing locally - https://phabricator.wikimedia.org/T318732 (10jgleeson) try this without '?debug=true' [21:11:05] 10Fundraising-Backlog: Declines in Italy for no 3DS - https://phabricator.wikimedia.org/T332950 (10AKanji-WMF) That's great - shall I close this task @EMartin [21:13:12] (03CR) 10Wfan: [C: 03+2] "If the parser will put these into to json when outputting then looks good to me, thanks 😊 I will keep that in mind too~" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/902218 (https://phabricator.wikimedia.org/T288585) (owner: 10Eileen) [21:23:26] 10Fundraising-Backlog, 10FR-dlocal, 10fr-donorservices: mask CVV digits on dLocal donation form - https://phabricator.wikimedia.org/T332890 (10AKanji-WMF) Also checking with dLocal as to whether they can enable [21:26:22] fr-tech with Andrew gone, anyone want to review this? https://gerrit.wikimedia.org/r/c/mediawiki/extensions/DonationInterface/+/903294 [21:26:52] ejegg: heheh I haven't finished for the day yet eh! [21:27:01] (03Merged) 10jenkins-bot: Remove more watchdogs from civicrm_module [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/902218 (https://phabricator.wikimedia.org/T288585) (owner: 10Eileen) [21:27:14] (though others are also welcome to take a look ofc :) ) [21:32:09] 10Fundraising-Backlog, 10FR-dlocal, 10fr-donorservices: mask CVV digits on dLocal donation form - https://phabricator.wikimedia.org/T332890 (10Ejegg) Looks like we can solve it for safari & chrome users with some CSS: `lang=css { -webkit-text-security: disc; text-security: disc; } ` https://stackover... [21:36:58] ah ok, thanks! [21:40:30] (03CR) 10AndyRussG: [C: 03+2] "Woohooo thanks!!" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/900717 (https://phabricator.wikimedia.org/T324297) (owner: 10Wfan) [21:41:10] Thanks Andy =D [21:41:35] (03PS2) 10Eileen: Remove usage of nasty api class rather than cleanup watchdog [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/902220 (https://phabricator.wikimedia.org/T288585) [21:43:18] :) [21:43:33] ejegg: were you able to get an CC numbers that don't reject on 3ds? [21:52:21] AndyRussG: we just brought that up in the meeting today - I don't think he's fixed it yet [21:53:01] ejegg: yeah just checking in case it was working already :) [21:53:12] (03Merged) 10jenkins-bot: Move PlaceholderFiscalNumber to dlocal_gateway [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/900717 (https://phabricator.wikimedia.org/T324297) (owner: 10Wfan) [21:56:03] (03PS5) 10Eileen: Cleanup to remove call to wmf_civicrm_get_civi_id [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/895871 (https://phabricator.wikimedia.org/T288585) [21:56:05] (03PS5) 10Eileen: Fully remove wmf_get_civi_id [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/895873 (https://phabricator.wikimedia.org/T288585) [21:56:07] (03PS2) 10Eileen: Remove more watchdogs from civicrm_module [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/902219 (https://phabricator.wikimedia.org/T288585) [21:56:09] (03PS3) 10Eileen: Remove usage of nasty api class rather than cleanup watchdog [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/902220 (https://phabricator.wikimedia.org/T288585) [21:59:30] I was just looking to see if I could do a push on watchdog & get it resolved this sprint - I rebased what was there -but of the rest I see some in `ContributionTrackingQueueConsumer::processMessage` - just trying to figure out if that would already be civicrm_initialized.... [22:04:12] (03PS1) 10Eileen: Remove watchdoc from queue2civicrm [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/903339 (https://phabricator.wikimedia.org/T288585) [22:06:15] 10Fundraising-Backlog, 10FR-dlocal, 10fr-donorservices: mask CVV digits on dLocal donation form - https://phabricator.wikimedia.org/T332890 (10Ejegg) >>! In T332890#8731532, @Ejegg wrote: > Looks like we can solve it for safari & chrome users with some CSS: > `lang=css > { > -webkit-text-security: disc;... [22:06:31] eileen: I'll take a peek [22:06:37] thanks AndyRussG [22:06:46] thank u! :) [22:06:53] ContributionTrackingQueueConsumer probs does not civicrm_init [22:06:56] eileen: [22:06:57] ^^ [22:07:09] since's it's purely hitting that drupal table [22:08:11] ejegg: ok- so I have been doing everything for completeness - which turned out handy when it comes to moving code - but that one specifically might not make sense to change? As it is not gonna just move to civi [22:09:01] eileen: hmm, right, so the whole table needs to move to the civicrm schema [22:09:03] oh hey found a neat trick for debugging JS in a browser from VSCode: just do control-shift-P then Debug: Open link and paste in a link to a local form [22:09:21] then you can set your breakpoints directly in your source code tabs in JSCode [22:09:25] VSCode [22:09:42] who, vscode runs its own browser? [22:09:59] ejegg: noo it spins up Chrome and remote-controls its debugger [22:10:04] ahhh [22:10:26] so you can still open the Chrome dev tools and see storage, network requests, etc. [22:10:59] (also VSCode is itself chromium-based [22:11:18] ) [22:22:27] 10fundraising-tech-ops, 10Infrastructure-Foundations, 10SRE, 10netops: Please update bootp helper on pfw3-eqiad to point to frpm1002 for fundraising subnets - https://phabricator.wikimedia.org/T332939 (10Dwisehaupt) Thanks! Verified working and runs good. [22:23:07] 10fundraising-tech-ops: reimage fundraising_database role servers to Bullseye - https://phabricator.wikimedia.org/T315601 (10Dwisehaupt) [22:23:31] 10fundraising-tech-ops: reimage fundraising_database role servers to Bullseye - https://phabricator.wikimedia.org/T315601 (10Dwisehaupt) 05Open→03Resolved a:03Dwisehaupt All db role servers running bullseye. [22:23:33] 10fundraising-tech-ops: Fundraising environment support for Debian 11 (Bullseye) - https://phabricator.wikimedia.org/T277184 (10Dwisehaupt) [22:28:45] 10fundraising-tech-ops: reimage fundraising bastion (frbast) role servers to Bullseye - https://phabricator.wikimedia.org/T333246 (10Dwisehaupt) [22:39:06] (03PS1) 10Eileen: Remove qatchdog from UnsubscribeQueueConsumer [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/903344 (https://phabricator.wikimedia.org/T288585) [22:43:32] (03CR) 10AndyRussG: [C: 03+2] "Works great, thanks!!! :) (See inline for question about possible follow-on task.)" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/903294 (https://phabricator.wikimedia.org/T332831) (owner: 10Ejegg) [22:44:32] (03PS2) 10Eileen: Remove watchdog from UnsubscribeQueueConsumer [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/903344 (https://phabricator.wikimedia.org/T288585) [22:44:34] (03PS1) 10Eileen: Remove watchdog from RefundQueueConsumer [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/903346 (https://phabricator.wikimedia.org/T288585) [22:45:12] (03Merged) 10jenkins-bot: Dlocal: Handle return from 3ds authentication [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/903294 (https://phabricator.wikimedia.org/T332831) (owner: 10Ejegg) [22:45:52] even though I know I'm getting there these watchdogs are a bit soul destroying [22:49:21] awww [22:53:47] keep the faith eileen ⌚ 🦮 [22:53:50] (03CR) 10AndyRussG: [V: 03+2 C: 03+2] "Thanks!!!" [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/902824 (https://phabricator.wikimedia.org/T324295) (owner: 10Wfan) [22:53:55] lol anilk [22:54:12] I think the dog is a bit more feral than that one [22:55:54] (03PS1) 10Eileen: Remove watchdog from WMFQueueConsumer [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/903347 (https://phabricator.wikimedia.org/T288585) [23:07:47] 10Fundraising-Backlog, 10FR-dlocal, 10fr-donorservices: dLocal audit file: some recent refunds not reaching Civi - https://phabricator.wikimedia.org/T333247 (10MBeat33) [23:12:22] ejegg|food: I feel like you said we don't even use `queue2civicrm_log` anymore? [23:36:47] (03PS1) 10Eileen: Remove watchdog from RecurringQueueConsumer [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/903349 (https://phabricator.wikimedia.org/T288585) [23:49:49] (03PS1) 10Eileen: Remove watchdog from civicrm_cleanup_old_deleted_contacts [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/903350 (https://phabricator.wikimedia.org/T288585) [23:56:58] (03PS1) 10Eileen: Remove unused function in test class [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/903351 (https://phabricator.wikimedia.org/T288585) [23:57:00] (03PS1) 10Eileen: Remove watching from recurring.module [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/903352 (https://phabricator.wikimedia.org/T288585)