05nelsonm / kmp-tor

Kotlin Multiplatform Library for embedding Tor into your applications
Apache License 2.0
33 stars 5 forks source link

`RuntimeEvent.ERROR` should always use `OnEvent.Executor.Immediate` #454

Closed 05nelsonm closed 1 month ago

05nelsonm commented 1 month ago

Did not note this, but reason for needing to use Immediate is to ensure that all ERROR observers can be notified of the exception using Handler.withSuppression. As it is currently implemented, the first ERROR observer to throw will stop the loop. They should be stacked into a single UncaughtException for ERROR and thrown afterwards.