[00:13:17] (03CR) 10Cstone: [C:03+2] Add parenthese for clarity [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1079029 (owner: 10Eileen) [00:14:55] https://yro.slashdot.org/story/24/10/09/2247234/internet-archive-suffers-catastrophic-breach-impacting-31-million-users [00:16:18] :( [00:23:04] yeah :( [00:23:38] that and the firefox security issue it was a busy day: https://www.bleepingcomputer.com/news/security/mozilla-fixes-firefox-zero-day-actively-exploited-in-attacks/ [00:28:46] 😔 [00:34:07] (03Merged) 10jenkins-bot: Add parenthese for clarity [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1079029 (owner: 10Eileen) [00:36:44] (03PS1) 10Eileen: Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1079043 [00:37:10] (03CR) 10Eileen: [C:03+2] Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1079043 (owner: 10Eileen) [00:38:20] (03Merged) 10jenkins-bot: Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1079043 (owner: 10Eileen) [00:38:48] ouch - I just closed ff & re-opened & now it is installing [00:41:06] !log civicrm upgraded from 3b6a7cbb to 07dee21c [00:41:07] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [00:55:23] 10fundraising-tech-ops, 06DC-Ops, 06Infrastructure-Foundations, 10netops, and 2 others: codfw:frack:servers migration task - https://phabricator.wikimedia.org/T375151#10216097 (10Papaul) We did phase 2 today, all the 1G nodes are now connected to the new fasw2-c8a/b. We will me moving the 10G nodes next we... [01:06:56] (03PS1) 10Eileen: Add preference tags placeholder [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1079046 [01:17:19] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Acoustic SMS stage 2 Data merge on email to acoustic is not updating name - https://phabricator.wikimedia.org/T376347#10216123 (10Eileenmcnaughton) @shust - you can go ahead & merge these contacts now - the bug is fixed, although getting the correct data... [01:58:03] (03CR) 10Cstone: [C:03+2] Add entity tags to test schema [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1079024 (owner: 10Eileen) [01:58:18] (03CR) 10Cstone: [C:03+2] Add preference tags placeholder [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1079046 (owner: 10Eileen) [01:59:43] (03Merged) 10jenkins-bot: Add entity tags to test schema [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1079024 (owner: 10Eileen) [01:59:43] (03Merged) 10jenkins-bot: Add preference tags placeholder [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1079046 (owner: 10Eileen) [03:16:03] (03PS2) 10Cstone: Switch proxy ports to config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1078011 (https://phabricator.wikimedia.org/T375492) [03:20:17] (03PS2) 10Cstone: Add in countries we accept the currency of already A-H Also alphabetized some [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1078783 (https://phabricator.wikimedia.org/T349667) [03:52:53] (03PS1) 10Cstone: Add in countries we accept the currency of already I-Z Also alphabetized [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1079052 (https://phabricator.wikimedia.org/T349667) [03:54:48] (03CR) 10CI reject: [V:04-1] Add in countries we accept the currency of already I-Z Also alphabetized [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1079052 (https://phabricator.wikimedia.org/T349667) (owner: 10Cstone) [03:56:44] (03PS2) 10Cstone: Add in countries we accept the currency of already I-Z Also alphabetized [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1079052 (https://phabricator.wikimedia.org/T349667) [06:19:09] (03CR) 10CI reject: [V:04-1] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] (REL1_42) - 10https://gerrit.wikimedia.org/r/1079082 (owner: 10L10n-bot) [06:35:38] (03PS1) 10Eileen: Do not uploade DatabaseUpdate file [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1079145 (https://phabricator.wikimedia.org/T368470) [06:36:26] (03CR) 10CI reject: [V:04-1] Do not uploade DatabaseUpdate file [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1079145 (https://phabricator.wikimedia.org/T368470) (owner: 10Eileen) [06:38:54] (03PS2) 10Eileen: Do not uploade DatabaseUpdate file [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1079145 (https://phabricator.wikimedia.org/T368470) [06:48:36] (03PS1) 10Eileen: WIP - mock code on adding folder search [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1079147 [06:52:34] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Permissions issue with php user accessing silverpop exports - https://phabricator.wikimedia.org/T376870 (10Eileenmcnaughton) 03NEW [06:52:57] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Permissions issue with php user accessing silverpop exports - https://phabricator.wikimedia.org/T376870#10216447 (10Eileenmcnaughton) p:05Triage→03High [11:13:19] 06Fundraising-Backlog: [low priority] Investigate conditional Google's script addition for Google Ads Grant (fundraising) - https://phabricator.wikimedia.org/T374079#10217179 (10MSuijkerbuijk_WMF) Hi @AKanji-WMF could we look into this? We are talking to the Legal team and they'd need some information from us li... [12:06:39] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Fundraiser Public Data Mover failed - https://phabricator.wikimedia.org/T376891 (10jgleeson) 03NEW [12:12:56] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Thank you mail send failed - https://phabricator.wikimedia.org/T376893 (10jgleeson) 03NEW [12:59:47] (03PS1) 10Jgleeson: Add civicrm update scripts to make life easier locally. [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1079275 [13:00:16] (03CR) 10Jgleeson: "@emcnaughton@wikimedia.org can you vet these please?" [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1079275 (owner: 10Jgleeson) [13:03:31] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Permissions issue with php user accessing silverpop exports - https://phabricator.wikimedia.org/T376870#10217578 (10Jgreen) I think this may be a coworker (running as user www-data) vs process-control (running as us... [13:07:04] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Fundraiser Public Data Mover failed - https://phabricator.wikimedia.org/T376891#10217589 (10Jgreen) 05Open→03Resolved a:03Jgreen This was due to work in our rack in codfw at the time. No action needed. [13:11:35] Jeff_Green: do we need to rerun that job? [13:11:47] it runs constantly [13:11:51] ah cool [13:11:56] well, peridocally and frequently [13:12:01] it's self-healing [13:12:16] :) [13:18:00] fr-tech. little tip when rebuilding civicrm. close phpstorm. I noticed when rebuilding earlier that phpstorm was getting worked up trying to reindex the files but as they are changing during the build steps, and this sent my laptop fans into overdrive. I shut it off and restarted it after the build complete and it indexed fine with no extra laptop heat! [13:19:22] lol good wikitech-l spam today [13:19:54] The Hair Loss Solution at ScalpmastersRI - we help restore confidence for those with thinning hair, baldness, alopecia, or receding hairlines [13:26:46] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog, 07payments-orchestration: Gravy PayPal Webhooks - https://phabricator.wikimedia.org/T376473#10217680 (10jgleeson) Confirm that Paypal Webhooks are coming in for captures and refunds. [13:28:58] 06Fundraising-Backlog, 07Epic, 07payments-orchestration: Gravy Paypal - https://phabricator.wikimedia.org/T376469#10217686 (10jgleeson) [13:29:29] anilk: do we put epics in sprint? I just noticed the gravy paypal epic was in sprint so took it out. some of its child tasks are still in sprint [13:29:51] thanks for doing that, I don't know how that happened! [14:13:55] 06Fundraising-Backlog, 06Content-Transform-Team-WIP, 06Language-Team, 10MediaWiki-extensions-CentralNotice, and 4 others: JavaScript building message boxes should now use mw.util.messageBox - https://phabricator.wikimedia.org/T375902#10217893 (10cscott) [15:02:35] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog, 07payments-orchestration: Display Gravy reconciliation ID in CiviCRM - https://phabricator.wikimedia.org/T374144#10218082 (10jgleeson) a:03jgleeson [15:31:57] fr-tech is it me or does that last email from Adyen contain a file in a weird format with seemingly encrypted data within it? I can't open it in anything other than text pad. [15:33:52] I think numbers is the mac excel [15:33:54] Yeah [15:35:06] oh right [15:35:12] I'll try opening it on my mac [15:53:21] (03PS1) 10Damilare Adedoyin: Do capture only for Gravy Trustly recurring charge [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1079311 [15:54:16] (03PS2) 10Damilare Adedoyin: Do capture only for Gravy Trustly recurring charge [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1079311 (https://phabricator.wikimedia.org/T371810) [16:07:44] (03PS3) 10Cstone: Add in countries we accept the currency of already A-H Also alphabetized some [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1078783 (https://phabricator.wikimedia.org/T349667) [16:07:47] (03CR) 10Ejegg: [C:03+2] Add in countries we accept the currency of already A-H Also alphabetized some [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1078783 (https://phabricator.wikimedia.org/T349667) (owner: 10Cstone) [16:16:28] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog: Add comms preferences footer to all Civi-generated emails - https://phabricator.wikimedia.org/T359445#10218556 (10AnnWF) >>! In T359445#10211089, @NNgu-WMF wrote: > @AnnWF is there a deadline for this task? @AKanji-WMF could you help with the deadli... [16:16:36] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Acoustic SMS stage 2 Data merge on email to acoustic is not updating name - https://phabricator.wikimedia.org/T376347#10218557 (10SHust) Thanks, @Eileenmcnaughton! [16:38:45] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog, 13Patch-For-Review, 07payments-orchestration: Gravy ACH (Trustly) - https://phabricator.wikimedia.org/T371810#10218652 (10Damilare) The Gravy-Trustly integration is blocked for the following reasons: - Trustly yet to provide production crede... [16:39:31] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog, 07payments-orchestration: Set up recurring charge for venmo - https://phabricator.wikimedia.org/T376122#10218654 (10Damilare) The Gravy-Braintree-Venmo integration is blocked for the following reasons: Gravy is working on the recurring transaction... [16:54:21] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 13Patch-For-Review: Check that Adyen supports countries we had available on Ingenico - https://phabricator.wikimedia.org/T349667#10218715 (10AMJohnson) Just flagging one more that I came across today. Donor in #1549449 located in Georgia received the sa... [17:02:45] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10Recurring-Donations, 13Patch-For-Review, 07recurring upgrade: Update text for recurring upgrade thank you email - https://phabricator.wikimedia.org/T371848#10218727 (10MSuijkerbuijk_WMF) Thanks, this is with @RLopez-WMF for updating. [17:08:45] (03PS3) 10Cstone: Add in countries we accept the currency of already I-Z Also alphabetized [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1079052 (https://phabricator.wikimedia.org/T349667) [17:09:01] (03CR) 10Ejegg: [C:03+2] Add in countries we accept the currency of already I-Z Also alphabetized [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1079052 (https://phabricator.wikimedia.org/T349667) (owner: 10Cstone) [17:33:00] (03Merged) 10jenkins-bot: Add in countries we accept the currency of already A-H Also alphabetized some [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1078783 (https://phabricator.wikimedia.org/T349667) (owner: 10Cstone) [17:33:56] (03Merged) 10jenkins-bot: Add in countries we accept the currency of already I-Z Also alphabetized [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1079052 (https://phabricator.wikimedia.org/T349667) (owner: 10Cstone) [17:43:48] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: 'Expected field payments_final_status bound for table payments_initial' failmails - https://phabricator.wikimedia.org/T374132#10218923 (10Ejegg) The latest instance of this happened even with Dami's fix deployed to SmashPig https://civicrm.wikimedia.org/... [17:49:38] (03PS1) 10Ejegg: Always set FinalStatus failed on no response status [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1079332 (https://phabricator.wikimedia.org/T374132) [17:51:36] huh, just got another of those final status missing failmails [17:51:49] so what the heck is going on with dd that makes it so much more likely to get those??? [17:52:29] ejegg: the ones I looked at were due to us not getting a response back from Adyen [17:52:39] damilare: reminded me yesterday [17:53:51] Oct 10 17:28:53 payments1006 adyen_gateway: 219653938:219653938.1 Unsuccessful createPayment response from gateway: : null [17:53:54] same again for that one [17:56:19] looks like minfraud didn't like that one either [18:01:13] yeah, they look odd for a couple reasons [18:01:42] well, this patch ought to make sure the smashpig response has a status, at least: https://gerrit.wikimedia.org/r/1079332 [18:02:39] I also see the same PHP session ID shared across a couple different contribution tracking IDs for the most recent failmail of that sort [18:03:14] grep 5fcqv2mfotfarml payments-adyen [18:04:29] are they still all ACH? [18:04:36] (03CR) 10Jgleeson: [C:03+2] "LGTM" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1079332 (https://phabricator.wikimedia.org/T374132) (owner: 10Ejegg) [18:04:37] this one was [18:05:17] (03Merged) 10jenkins-bot: Always set FinalStatus failed on no response status [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1079332 (https://phabricator.wikimedia.org/T374132) (owner: 10Ejegg) [18:07:47] this gives a slightly more complete picture, but the sort might disorder some things within a single second: egrep '219653544|219653938|219654060' * | sort -k3 [18:08:14] (using the sort to get the bits from different files interleaved more or less correctly) [18:15:56] there are some 'Initiating cURL' lines that don't have any ctid prefix, at the timestamp of the failed call [18:18:43] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Permissions issue with php user accessing silverpop exports - https://phabricator.wikimedia.org/T376870#10219083 (10Eileenmcnaughton) @Jgreen hmm - I was running just on the command line - so perhaps it would have b... [18:19:33] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 13Patch-For-Review: 'Expected field payments_final_status bound for table payments_initial' failmails - https://phabricator.wikimedia.org/T374132#10219095 (10Ejegg) More odd stuff looking at this failed ACH from very recently: https://civicrm.wikimedia.... [18:22:08] can I get someone to merge this https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/tools/+/1079145 - then I'll run silverpop-daily to upload the other 4 files [18:24:28] I think I'll upload the full upload from last night the new way now to check that all works [18:25:07] I guess the only thing that could really go wrong is the fields go the wrong place but I don't really see how.... [18:26:49] (03PS1) 10Eileen: Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/tools into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1079336 [18:27:03] (03CR) 10Eileen: [C:03+2] Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/tools into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1079336 (owner: 10Eileen) [18:27:25] (03Merged) 10jenkins-bot: Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/tools into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1079336 (owner: 10Eileen) [18:29:19] !log tools upgraded from e9c05e30 to 632bf430 [18:29:21] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [18:31:52] cstone: how are your parents [18:34:04] (03CR) 10Jgleeson: [C:03+2] "LGTM!" [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1079145 (https://phabricator.wikimedia.org/T368470) (owner: 10Eileen) [18:34:46] thanks jgleeson [18:35:00] (03Merged) 10jenkins-bot: Do not uploade DatabaseUpdate file [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1079145 (https://phabricator.wikimedia.org/T368470) (owner: 10Eileen) [18:35:12] eileen: I added these earlier when trying to upgrade civi locally https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/dev/+/1079275 [18:35:46] do we still need to run the drush updb or is that redundant now ? [18:36:48] jgleeson: ok cool - FYI - we will need to switch to using cv rather than drush at some point - I just tend to fall back on drush cos you don't need to be in the right directory if you use the alias (we have wmf-cv on prod which is mostly like the alias) [18:37:15] yeah I was thinking drush my be old hat [18:37:20] updb is for the drupal upgrades but they won't really happen now [18:37:39] can you remap one of those to using cv and just paste it here and then I'll update the patch [18:37:44] (03CR) 10Eileen: [C:03+2] Add civicrm update scripts to make life easier locally. [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1079275 (owner: 10Jgleeson) [18:37:55] or not [18:37:57] :) [18:38:15] s/my/might/ [18:38:40] damilare: 3D Secure is switched on for UK payments on Gravy live. I just tried two payments and they worked [18:38:48] https://payments.wikimedia.org/index.php?title=Special:GravyGateway&appeal=JimmyQuote&country=GB¤cy=GBP&payment_method=cc&recurring=0&uselang=en-gb&amount=2&opt_in=0&wmf_medium=spontaneous&wmf_campaign=spontaneous&wmf_source=fr-redir.default~default~default~default~control.cc&wmf_key=otherAmt_0~time_5&referrer=www.google.com%2F [18:39:12] jgleeson: oh I wasn't gonna hold it up - for the api ones it would be a straight replace of drush @wmff cvapi with cv api4 [18:39:43] I'll do it now while it's fresh [18:40:10] for some of others I think I would need to check help (ie upgrade) [18:40:25] lemme try mine too jgleeson [18:40:27] you can also pass in user [18:40:54] damilare: it's only enabled for amex at the moment [18:41:03] ohh ok [18:41:23] yh but we can also test out the none 3ds flow to be sure it isn't broken [18:41:42] did you already test that [18:41:44] ? [18:41:53] no I was hoping someone from the US could test [18:42:08] but I guess we could also check the visa / non-3ds [18:42:31] (03PS1) 10Eileen: Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/tools into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1079338 [18:42:46] (03CR) 10Eileen: [C:03+2] Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/tools into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1079338 (owner: 10Eileen) [18:43:07] (03Merged) 10jenkins-bot: Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/tools into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1079338 (owner: 10Eileen) [18:43:30] mine failed jgleeson e [18:43:44] yeah mine too [18:43:45] mastercard [18:43:51] I tried Visa [18:44:02] hmmm lemme check the logs [18:44:28] !log tools upgraded from 632bf430 to 62f2d170 [18:44:29] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [18:44:37] it's just "declined" on the dashboard [18:44:50] "error_data":{"code":"303","description":"Access denied, invalid endpoint.","detail":" Acquirer not participating " [18:44:52] and we don't have the api logs on the dash anymore? [18:45:35] hmm is this the NPI network issue he was talking about [18:45:46] that's not good [18:45:47] just asking now [18:47:09] oh wait there [18:47:11] Eileen they are good no damage at the house [18:47:13] I think I know why [18:47:29] let's turn off all the other card networks [18:47:32] cstone: phew - I have been worrying about them & I don't even know them :-) [18:47:54] oh man [18:47:57] it clears all the settings [18:48:09] that's painful. took me ages to add those [18:48:19] They were the least worried out of all of us [18:48:27] Nuuu jgleeson [18:48:34] Ok dami mastercard and visa should now *not* be 3D Secure [18:48:38] I'm out walking the dog but I can test 3ds when I get back if needed [18:48:40] damilare: * [18:48:45] thanks cstone ! [18:48:49] ok I'd try again [18:49:34] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: 'Expected field payments_final_status bound for table payments_initial' failmails - https://phabricator.wikimedia.org/T374132#10219161 (10Ejegg) Steps to replicate locally: * try doing an ACH on Adyen with the correct test credentials (any name, any acct... [18:49:53] it workd! [18:49:58] with an e [18:50:11] OK, I figured out what's happening. But the fix will take some careful consideration... [18:50:14] "three_d_secure":null [18:50:18] I think I may have hit the fix [18:50:22] filter* [18:50:27] ahh [18:50:31] what's your ct_id? [18:50:36] 219664940.2 [18:51:39] "intent_outcome":"succeeded" [18:51:41] weird [18:52:07] so I kicked off our main nightly upload via the command line [18:52:11] https://usercontent.irccloud-cdn.com/file/hDdYdnUN/image.png [18:52:38] ah yeah damilare. fraud fail [18:52:43] yep [18:52:50] AVS and UTM [18:52:59] I think it would work though [18:53:04] my visa one got through [18:53:17] yea the last one didn't get authorized at all [18:53:18] https://wikimedia.gr4vy.app/merchants/default/transactions/da7407f7-a689-4f7f-8dc9-ee22c10df266/overview [18:53:20] so this is good [18:53:30] yep! [18:53:52] you only had to remove mastercard/visa from 3ds right [18:54:15] do we need to manually set those card types [18:57:58] yep damilare I deleted the settings for those methods [18:58:16] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog, 10FR-email-preferences: Email pref centre - Solve email overwrite issue from recurring - https://phabricator.wikimedia.org/T353376#10219213 (10AnnWF) Adding braintree venmo here, they have an IPN https://developer.paypal.com/braintree/articles/guid... [18:58:19] although I guess we also need to test it when configured [18:58:22] and from the US [18:58:43] yep [18:59:47] I think we're gonna have to set up the routing config to send applicable countries down the 3DS path [19:00:07] I had to add a rule for the UK here https://wikimedia.gr4vy.app/merchants/default/flow/card-transaction/skip-3ds/17d741a3-b531-444e-b64e-b2a91cce732c [19:00:27] eileen: is that upload with the Omnimail Upload ??? [19:00:31] so I guess the model will be, enable it for all, and add a "flow" rule for each country we want to send down it [19:00:53] ejegg: the screenshot - yest [19:00:58] woohoo! [19:00:59] which is easy enough for all methods except AMEX [19:01:07] yea that sounds about right [19:01:11] so it's a good time to merge that lil python change eileen ? [19:01:27] that one needs 5-6 individual "Connections" each with their own rule [19:01:28] I'm trying to see if Natalie or Brian will QA but it seems pretty safe because it relies on the csv file column names matching Acoustic [19:01:36] due to that unique MID situation [19:01:39] I think the python change is merged [19:01:41] though that rule you just shared was on curency not country jgleeson [19:01:43] Oh i see, jgleeson already C+2ed it [19:01:50] cool cool, exciting times [19:02:28] but the remaining todo is to turn this into a real boy https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/1079147 [19:02:31] yeah that was currency damilare [19:03:05] to replace the stuff that the python script does - basically looking in the folder, finding matching files & choosing the most recent [19:03:53] & also on that - I don't think it's great practice that full filenames & directories are accepted by the api - I just hadn't thought through the best way to limit it [19:04:32] I'm going to re-enable silverpop-daily now - that will mean generation & uploading of the other files is back on [19:06:43] !log config revision changed from ae4a5be9 to 150b02a9 [19:06:45] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [19:08:43] ok the job is back on so we have scheduled generating all the files & uploading the other files but the main upload is not scheduled until https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/1079147 is done - if anyone would love to code that jump in - otherwise I'll try to do it once I've done some morning stuff [19:09:03] jgleeson: damilare did you still need testing im back now [19:11:49] cstone: do you have amex [19:11:50] eileen: I can jump in and translate the old python to php for that one [19:12:07] oh just sorting should be enough [19:12:24] jgleeson do but its not at this house /unhelpful [19:12:28] I do* [19:12:30] ha [19:12:54] hmm ok I haven't got the visa 3ds rules in play. I just deleted them all :( [19:13:02] I wanted to confirm the pass-through was working [19:13:43] I'll add visa rules back in and then we can confirm that UK traffic from damilare and I get's 3d secure and US traffic gets the usual [19:18:50] 06Fundraising-Backlog, 10fundraising-tech-ops: FR-Tech FY2425Q2 maintenance window (Nov 4-8th, 2024) - https://phabricator.wikimedia.org/T337583#10219281 (10greg) [19:20:45] fr-tech is there a good place to put all these 3d secure settings that we can all access? do we have a file server location or has that thing been replaced with google docs [19:21:10] (03PS2) 10Jgleeson: Add civicrm update scripts to make life easier locally. [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1079275 [19:21:50] eileen: I've updated the scripts to all use cv and dropped drush https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/dev/+/1079275 [19:23:06] jgleeson: it's a 'google shared drive' now.... [19:23:16] oh right [19:23:18] somehow different from just google docs i guess? [19:24:31] I don't think I have access to that thing yet so I'll create a google spreadsheet and invite us all and then export it to there once I figure out access [19:25:33] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog: Add comms preferences footer to all Civi-generated emails - https://phabricator.wikimedia.org/T359445#10219325 (10AKanji-WMF) Hi @AnnWF - Mariana flagged that this raises the question the page that these links will drive to - our comms pref centre. I... [19:31:07] (03PS2) 10Ejegg: Upload latest CSV file in a folder [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1079147 (https://phabricator.wikimedia.org/T368470) (owner: 10Eileen) [19:31:24] eileen: ^^^ seems to work for me [19:33:32] ejegg: wow just like magic - I come back & it is done! [19:34:55] * ejegg takes off elf hat [19:36:17] :-) [19:36:44] oh wow - just a plain sort! [19:36:55] that's what the python is doing, right? [19:37:21] orrr... was it more complicated? [19:38:44] ahh, it was the latest created date [19:38:54] ok, I can make it match that eileen [19:39:19] probably not as succinctly as max(glob.iglob(updatesglob), key=os.path.getctime) though [19:39:26] ejegg: well - sort is possible fine though? [19:39:38] for our purposes it should be [19:40:10] do you have thoughts about any hardening we should do - my best idea at the moment is to have a setting that constrains the possible paths (set via civicrm.settings.php) [19:41:19] ah yeah, prefix=passwd sourceFolder=/etc [19:42:11] I'll add that [19:43:59] ejegg: yeah it's a problem across the pre-existing ones too - ie csfFile,xmlFile [19:44:09] I guess we should check that source folder in both cases - when explicitly specifying the file and when scanning a dir [19:44:19] yep [19:45:00] so say we have a setting 'permitted_folders' then we could check in there, but maybe allow it not to be set for dev? [19:45:55] ah I was going to add it to the dev settings [19:46:13] oh but that might be annoying for running tests I guess? [19:47:15] i mean, we need omnimail_credentials to be set for stuff to work, right? [19:47:43] I'll do the folder restriction in a follow-on commit [19:56:58] (03PS1) 10Eileen: Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1079351 [19:57:09] (03CR) 10Eileen: [C:03+2] Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1079351 (owner: 10Eileen) [20:00:19] oh wait, I should set $this-csvFile after that scan, not just return [20:00:47] ejegg: oh yeah - opps I just checked the selection [20:01:10] well it will work as is - just less consistent [20:01:20] & slightly more file scans (2 vs 1 I thnk) [20:02:57] (03PS1) 10Ejegg: One less file scan [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1079352 [20:03:06] ejegg: on the Acoustic sftp uploads folder there are matching gifts upload files going back to late Sep - do you think we stopped doing them? [20:03:34] (03CR) 10Eileen: [C:03+2] One less file scan [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1079352 (owner: 10Ejegg) [20:04:02] ah yeah, I think we did stop that - we're uploading the matching gift info right in the contact rows now [20:04:03] (03CR) 10Eileen: [C:03+2] Add civicrm update scripts to make life easier locally. [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1079275 (owner: 10Jgleeson) [20:04:12] so that separate file wasn't needed I believe? [20:05:21] !log civicrm upgraded from 07dee21c to ff3144dd [20:05:23] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [20:05:38] ejegg: for MatchingGifts? I'm confused - do we stop uploading it? [20:06:08] let me see if I can find something in the process-control logs [20:07:24] ty eileen ! [20:08:17] eileen: ok, i don't see any change related to that [20:08:26] but I remember thinking that it could be discontinued [20:09:02] (03CR) 10Jgleeson: [V:03+2] Add civicrm update scripts to make life easier locally. [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1079275 (owner: 10Jgleeson) [20:11:53] ejegg: can you check the commit in process control - it worked on prod [20:21:45] sorry eileen, just looking now [20:26:03] looks fine to me eileen [20:26:15] thanks ejegg [20:27:29] !log config revision changed from 150b02a9 to 3c6d2054 [20:27:31] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [20:28:24] (03PS1) 10Cstone: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1079357 [20:30:22] ah the xml mapping file is only created in the tmp dir, so we only need to check the csv file for allowed dirs [20:31:15] (03CR) 10Cstone: [C:03+2] Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1079357 (owner: 10Cstone) [20:31:33] (03Merged) 10jenkins-bot: One less file scan [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1079352 (owner: 10Ejegg) [20:42:03] ejegg: well that is if the mapping file is not passed in - which we might do in future - although we might put it in a different place (eg. checked in for Unsubscribes - which can't just be generated) [20:43:45] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Acoustic SMS stage 2 Switch Acoustic mapping to use the API to define the columns - https://phabricator.wikimedia.org/T368470#10219609 (10Eileenmcnaughton) We seem to be successfully uploading our main DatabaseUpdat... [20:47:22] oh i see eileen [20:48:29] (03CR) 10Cstone: [C:03+2] Css for MC button in smaller screen [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1078503 (https://phabricator.wikimedia.org/T375545) (owner: 10Wfan) [20:49:51] (03CR) 10Cstone: [C:03+2] Log fund failed with reason for adyen [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1076056 (https://phabricator.wikimedia.org/T373555) (owner: 10Wfan) [20:50:17] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Check on outcome of upload job - https://phabricator.wikimedia.org/T376950 (10Eileenmcnaughton) 03NEW [20:50:18] (03Merged) 10jenkins-bot: Log fund failed with reason for adyen [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1076056 (https://phabricator.wikimedia.org/T373555) (owner: 10Wfan) [20:51:52] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1079357 (owner: 10Cstone) [20:51:53] (03Merged) 10jenkins-bot: Css for MC button in smaller screen [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1078503 (https://phabricator.wikimedia.org/T375545) (owner: 10Wfan) [21:05:29] (03PS1) 10Ejegg: WIP Only allow uploading files in certain directories [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1079359 [21:05:50] I think that's about right eileen, just need to smoke test it [21:06:55] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog: Add comms preferences footer to all Civi-generated emails - https://phabricator.wikimedia.org/T359445#10219724 (10Cstone) @AKanji-WMF that's what this ticket is about, the translations of the preference center itself {T376514} [21:07:58] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog: Placeholder - Figure out how translations of the email preference center will work - https://phabricator.wikimedia.org/T376514#10219739 (10Cstone) [21:09:27] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog: Add comms preferences footer to all Civi-generated emails - https://phabricator.wikimedia.org/T359445#10219748 (10AnnWF) >>! In T359445#10219324, @AKanji-WMF wrote: > Hi @AnnWF - Mariana flagged that this raises the question the page that these links... [21:12:41] need to pick up kid, will try to get back online later [21:27:25] (03CR) 10CI reject: [V:04-1] WIP Only allow uploading files in certain directories [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1079359 (owner: 10Ejegg) [21:30:08] 06Fundraising-Backlog: Foundation logo overlapping monthly convert modal in smaller screen sizes - https://phabricator.wikimedia.org/T374074#10219826 (10AnnWF) a:03AnnWF [21:30:22] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Foundation logo overlapping monthly convert modal in smaller screen sizes - https://phabricator.wikimedia.org/T374074#10219829 (10AnnWF) [21:31:04] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: change Paypal descriptor to omitt US and CA reference that is confusing donors - https://phabricator.wikimedia.org/T376077#10219840 (10AnnWF) a:03AnnWF [21:47:48] 06Fundraising-Backlog, 10FR-donorservices: Possible issue with the civi snooze - https://phabricator.wikimedia.org/T376959 (10SHust) 03NEW [22:00:34] 06Fundraising-Backlog, 10FR-donorservices: Snooze date is gone from summary page - https://phabricator.wikimedia.org/T376962 (10SHust) 03NEW [22:01:02] 06Fundraising-Backlog, 10FR-donorservices: Snooze date is gone from summary page - https://phabricator.wikimedia.org/T376962#10219963 (10SHust) [22:02:26] I haven't looked at ^^ but it snooze seemed to be working when I looked the other day - I'm heading out for a while now but note that [22:02:26] 1) snooze date will be missing, if empty on a contact [22:02:26] 2) it should show up in the in-line edit now [22:37:02] !log payments-wiki upgraded from ebb42c67 to 40e4a592 [22:37:03] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [22:38:16] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10FR-Tech-Analytics, 10FR-tech-data-integrity: New Donors Donating On Email - https://phabricator.wikimedia.org/T356911#10220046 (10Cstone) a:05Cstone→03None [22:44:26] (03PS1) 10Wfan: Foundation logo overlapping monthly convert modal in smaller screen sizes [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1079367 (https://phabricator.wikimedia.org/T374074) [22:46:23] (03CR) 10CI reject: [V:04-1] Foundation logo overlapping monthly convert modal in smaller screen sizes [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1079367 (https://phabricator.wikimedia.org/T374074) (owner: 10Wfan) [22:49:08] (03PS2) 10Wfan: Foundation logo overlapping monthly convert modal in smaller screen sizes [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1079367 (https://phabricator.wikimedia.org/T374074) [23:18:31] cstone [23:19:09] Hi do you know why we are not working on the dlocal recurring descriptor? I still see we use wikimedia foundation for all recurring now [23:20:12] hmmmm I can't remember, I thought it was showing up in the dlocal console now? [23:20:46] Description [23:20:46] Wikimedia Foundation (Recurring) [23:21:00] just looked at a random one that was the 8th charge [23:22:32] ha, ok I think I saw 2022 [23:22:56] seems works for 2024 haha, then do you know if we have dynamic descriptors for paypal [23:23:04] I am looking at this ticket https://phabricator.wikimedia.org/T376077 [23:23:44] yeah paypal is doing something different completely I think? [23:24:13] theres the one time descriptor, the descriptor set by the recurring charge job and then im not sure what paypal is doing on recurrings or even on their end [23:24:19] seems like we need to first have dynamic descriptors enabled, then depends on processor, we can set it up from console [23:25:37] Yeah, and since I do not have the paypal console, so not sure this init and recurring descriptor😅 [23:26:02] oh but I think I have one for sandbox, nvm I will test it there [23:43:47] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: change Paypal descriptor to omitt US and CA reference that is confusing donors - https://phabricator.wikimedia.org/T376077#10220268 (10AnnWF) See from paypal that https://www.paypal.com/us/cshelp/article/why-is-the-number-402-935-7733-showing-on-my-bank-o...