I think most users of the library will just log an error and ignore the InteractError anyways. The default behavior should be to escalate the error rather than trying to handle it "gracefully".
For those people that really know what they're doing and need to handle panics an additional interact_safe could be added which does behave exactly like before.
I'm interested to hear from users of this library if they ever had a need for handling a panic (or abortion) after an interact call.
The return type of
interactis questionable at best:InteractError::Panicis dangerous to ignoreInteractError::Abortedwill never be created when using theSyncWrapperinterfaceI'm proposing to remove this Result from the call signature
What happens in the error cases?
It just panics. :boom:
Why is this a good thing?
I think most users of the library will just log an error and ignore the
InteractErroranyways. The default behavior should be to escalate the error rather than trying to handle it "gracefully".For those people that really know what they're doing and need to handle panics an additional
interact_safecould be added which does behave exactly like before.I'm interested to hear from users of this library if they ever had a need for handling a panic (or abortion) after an interact call.