Currently if the webhook handler functions the user of this library defines throw an error, this breaks the entire application stack including the agent.
This is bad. Probably the agent should also not break like this, but we can and should handle this in the basic controller by wrapping this in a try, except code block. This is probably something we should look at throughout the code.
How to Reproduce
Run any of the docker compose setups in tutorial or projects
Open one of the notebooks
Define a handler function that intentionally errors
Cause that handler function to be triggered (most easiest is the connection handler which is triggered through the create_invitation() function)
Expected Behavior
We should handle the error gracefully, this is an easy but important fix.
Description
Currently if the webhook handler functions the user of this library defines throw an error, this breaks the entire application stack including the agent.
This is bad. Probably the agent should also not break like this, but we can and should handle this in the basic controller by wrapping this in a try, except code block. This is probably something we should look at throughout the code.
How to Reproduce
Expected Behavior
We should handle the error gracefully, this is an easy but important fix.