kpgalligan
04/30/2020, 1:02 PMphldavies
04/30/2020, 1:09 PMnetwork and removed the withContext wrapper - seems to work finephldavies
04/30/2020, 1:11 PMcoroutineScope too and just leave the MainThread guard in place - will the coroutineScope have any impact on cancellation?kpgalligan
04/30/2020, 1:15 PMrusshwolf
04/30/2020, 1:17 PMcoroutineScope was to try and play nice with typical coroutine semantics if you wanted to add a launch internally or something. But realistically that probably just makes it easier to do something that won't work.phldavies
04/30/2020, 2:48 PMcoroutineScope in the network function on Android would at least keep the coroutine semantics better aligned between the two and doesn’t exhibit the deadlock I was seeing on Android with both the coroutineScope and the withContext(childContext())