[17:21:11] mdholloway: There's now an MCS patch to remove the titles in the href from reference links: https://gerrit.wikimedia.org/r/#/c/332423/. I also have another patch to fix the tests. [17:23:53] bearND: cool, i'll have a look [19:41:56] niedzielski: dbrant: account creation seems to be broken on all versions of the app. looks like captcha retrieval is messed up somehow. [19:42:14] :.( [19:43:13] geez, i can repro that :| [19:44:05] looks like the url from authmanager may have changed... although it would be odd for that suddenly to go live on all wikis at the same time, without notice... [19:45:39] nope, that's not it [19:45:44] i cannot create an account on wikipedia.org? [19:46:26] ah, capcha got me twice! [19:46:52] i've gotta switch IPs, getting throttled now :( [19:54:38] mdholloway: around? [19:54:53] jdlrobson: o/ [19:55:09] So was about to merge your patch https://gerrit.wikimedia.org/r/#/c/325044/ but noticed apihelp-query+pageimages-param-prop needs updating [19:55:14] so it shows up in http://localhost:8888/w/index.php/Special:ApiSandbox#action=query&format=json&prop=pageimages&titles=Hannah+Montana&pithumbsize=50 [19:55:29] are you able to quickly fix that before i merge? [19:55:52] jdlrobson: sure, i'll have a look [19:58:03] mdholloway: is there a ticket for the account creation bug or should i make one? [19:59:42] niedzielski: not yet afaik, have been trying to quickly track the cause down but no luck so far, if you wouldn't mind making a phab that would be splendid! [20:00:09] mdholloway: sure thing. thanks! i'll cc you on it [20:13:44] niedzielski: were you testing manually when you were looking at my account creation series a couple hours ago? i know account creation was working as recently as yesterday afternoon [20:29:04] jdlrobson: patch updated! https://gerrit.wikimedia.org/r/#/c/325044/ [20:33:42] mdholloway: i don't believe i tried of the new patches so i think that's independent of this issue [21:21:55] mdholloway: on it [21:22:48] mdholloway: doesnt seem to be showing on https://en.wikipedia.org/wiki/Special:ApiSandbox#action=query&format=json&prop=pageimages ? [21:22:54] ahh en.wikipedia. [21:22:56] not localhost [21:55:58] mdholloway: i saw your commonet on T155535. do you think this is an image lib / cookie issue? it seems more like the initial request for a captcha is misunderstood [21:55:58] T155535: Account creation fails - https://phabricator.wikimedia.org/T155535 [21:57:20] niedzielski: i'm really not sure. it does seem to be fresco-specific. beyond that...? [21:57:44] fresco's logging isn't particularly helpful, either [21:57:49] mdholloway: on my device, i don't even get a broken image. i just get the snackbar and never even attempt to show an image [22:00:44] mdholloway: actually, the only place i see captchas even being requested is in the onclick listener for the refresh captcha button [22:02:38] niedzielski: yeah, fresco's own requests don't show up in logcat (assuming it's happening here at all -- i played a little bit with fresco's logging as directed at http://frescolib.org/docs/troubleshooting.html#using-the-logcat but not much help) [22:10:29] mdholloway: i tried the fresco patch on top of master with a local vagrant install (with captcha) and it still seems to fail for me [22:10:54] niedzielski: the picasso patch? [22:11:18] er, sorry, yes. this guy: https://gerrit.wikimedia.org/r/#/c/332647/ [22:27:31] niedzielski: i just verified again that the picasso patch loads the captcha on prod enwiki [23:42:06] niedzielski: well here's something interesting [23:42:28] seeing this, i decided to try to repro on a api 16 nexus S emulator: https://github.com/facebook/fresco/issues/1041 [23:42:41] and captcha loading works fine in that emulator [23:42:47] so we might have something device specific [23:42:50] neat! [23:42:55] if i set the image as initially visible, it doesn't load but re-tapping does load it [23:42:55] I've been on my 6P [23:43:47] me too [23:43:57] i was wondering if it was some variation of this though: https://github.com/facebook/fresco/issues/1430 (not quite the same thing) [23:45:09] niedzielski: also, via a random redditor: "P.S. The way to use Fresco "wrong" is by using it." [23:45:18] lol [23:45:30] :] [23:48:00] niedzielski: i'm trying to get over a post-travel cold so I think I'm going to call it a day, but at least it looks like we have some good leads. [23:48:27] mdholloway: sounds good. see ya tomorrow! [23:48:32] btw, what i'm trying to figure out is whether images will load if their visibility is GONE [23:49:05] niedzielski: ooh... good question. [23:49:30] niedzielski: see you tomorrow! [23:49:36] o/