Rethrow CancellationException in try-catch blocks for coroutines

[Obolrom]

Here is a code from repository LoginVM

The CancellationException should be re-thrown in a catch block when we use try-catch in coroutines.

Why?

Kotlin doc
Medium article
Florina Muntenescu's article - Cancellation in coroutines

[Obolrom]

If I am not mistaken, this can lead to UI bugs

The problem here is we do not handle CancellationException. In the catch block, the consumeException is called (the implementation)
Thus, alertFactory.unknownErrorAlert(closable, e = exception) line will be called. However, the cancellation event should not be the trigger for showing an alert

My proposition here at least do not call consumeException(e, templateKey) and errorIndicator?.postValue(true) if CancellationException has occurred, to avoid showing an unnecessary alert