app-common/src/main/java/im/angry/openeuicc/service/EuiccChannelManagerService.kt

@@ -14,7 +14,6 @@ import im.angry.openeuicc.common.R
 import im.angry.openeuicc.core.EuiccChannelManager
 import im.angry.openeuicc.util.*
 import kotlinx.coroutines.Dispatchers
-import kotlinx.coroutines.NonCancellable
 import kotlinx.coroutines.delay
 import kotlinx.coroutines.flow.Flow
 import kotlinx.coroutines.flow.MutableSharedFlow
@@ -25,7 +24,6 @@ import kotlinx.coroutines.flow.onCompletion
 import kotlinx.coroutines.flow.onStart
 import kotlinx.coroutines.flow.takeWhile
 import kotlinx.coroutines.flow.transformWhile
-import kotlinx.coroutines.isActive
 import kotlinx.coroutines.launch
 import kotlinx.coroutines.withContext
 import kotlinx.coroutines.withTimeoutOrNull
@@ -196,7 +194,7 @@ class EuiccChannelManagerService : LifecycleService(), OpenEuiccContextMarker {
             updateForegroundNotification(title, iconRes)
 
             try {
-                withContext(Dispatchers.IO + NonCancellable) { // Any LPA-related task must always complete
+                withContext(Dispatchers.IO) {
                     this@EuiccChannelManagerService.task()
                 }
                 // This update will be sent by the subscriber (as shown below)
@@ -206,11 +204,9 @@ class EuiccChannelManagerService : LifecycleService(), OpenEuiccContextMarker {
                 Log.e(TAG, Log.getStackTraceString(t))
                 foregroundTaskState.value = ForegroundTaskState.Done(t)
             } finally {
-                if (isActive) {
                 stopSelf()
             }
         }
-        }
 
         // We should be the only task running, so we can subscribe to foregroundTaskState
         // until we encounter ForegroundTaskState.Done.

app-common/src/main/java/im/angry/openeuicc/ui/ProfileDownloadFragment.kt

@@ -215,24 +215,20 @@ class ProfileDownloadFragment : BaseMaterialDialogFragment(),
         lifecycleScope.launch {
             ensureEuiccChannelManager()
             euiccChannelManagerService.waitForForegroundTask()
-            val res = doDownloadProfile(server, code, confirmationCode, imei)
+            try {
-
+                doDownloadProfile(server, code, confirmationCode, imei)
-            if (res == null || res.error != null) {
+            } catch (e: Exception) {
                 Log.d(TAG, "Error downloading profile")
-
+                Log.d(TAG, Log.getStackTraceString(e))
-                if (res?.error != null) {
+                Toast.makeText(context, R.string.profile_download_failed, Toast.LENGTH_LONG).show()
-                    Log.d(TAG, Log.getStackTraceString(res.error))
+            } finally {
-                }
-
-                Toast.makeText(requireContext(), R.string.profile_download_failed, Toast.LENGTH_LONG).show()
-            }
-
                 if (parentFragment is EuiccProfilesChangedListener) {
                     (parentFragment as EuiccProfilesChangedListener).onEuiccProfilesChanged()
                 }
                 dismiss()
             }
         }
+    }
 
     private suspend fun doDownloadProfile(
         server: String,
@@ -258,7 +254,7 @@ class ProfileDownloadFragment : BaseMaterialDialogFragment(),
             }
         }.last()
 
-        res as? EuiccChannelManagerService.ForegroundTaskState.Done
+        (res as? EuiccChannelManagerService.ForegroundTaskState.Done)?.error?.let { throw it }
     }
 
     override fun onDismiss(dialog: DialogInterface) {