[09:21:54] o/ [17:53:40] dbrant: i'm still seeing a similar error after running the tests locally. not sure if it's the same issue or something different: https://phabricator.wikimedia.org/P4232 [17:54:02] dbrant: i'm looking into it but if you spot the problem, let me know! [17:54:11] niedzielski: checking... [17:56:37] dbrant: hm, actually, i'm wondering if this is a proguard issue now that i'm looking at the raw logcat. sorry, that paste was taken from a logcat regex i normally use and wasn't complete. i'll update it [17:58:03] dbrant: there are some VFY complaints i'd like to check against a couple releases ago [17:58:10] (on API 16) [20:11:29] dbrant: i've seen the issue happen just by launching the app but it's very sporadic. it seems to be independent of proguard after all as i had it happen once on a dev build [20:11:59] niedzielski: well shoot. [20:12:25] devdebug that is [20:12:52] niedzielski: "don't keep activities" enabled or no? [20:13:20] dbrant: ah, looks like i have that on [20:13:58] niedzielski: i'll try thrashing the app to reproduce, hopefully. [20:14:54] dbrant: i can't seem to repro it regularly. the couple times it has happened, i haven't been able to introduce any debugging [20:15:54] niedzielski: :/ what can possibly lead to the Client being initialized before app.onCreate()?? [20:16:47] dbrant: not sure. maybe initalization at declaration / static members or possibly a service occurring at the same time as onCreate? [20:17:03] dbrant: i was wondering if it had something to do with the cookie manager [20:17:47] dbrant: the npe is on "cookieHeaders = cookieHandler.get(url.uri(), headers);" in JavaNetCookieJar [20:18:21] right, meaning that a null cookieManager was passed into it upon initialization [20:19:02] android lifecycles (╯°□°)╯︵ ┻━┻ [20:56:20] dbrant: so we can merge that new patch in but i don't have a great way to verify it. i've been trying to repro under different network conditions here but it's just not happening [21:28:08] niedzielski: if you're up for it, let's do a beta with that patch [21:30:13] dbrant: okey dokey [22:21:54] dbrant: according to the google console pre-launch testing results are currently delayed. i don't think they're worth blocking for but thought i'd ask for a second opinion [23:02:20] i guess no that delayed. results are good [23:04:04] ok 160 is out