Currently if a webhook gives up on an item for whatever reason (e.g. channel TTL, webhook TTL, max attempts reached) a message is added to the error array stored in Zookeeper for that webhook. I am thinking we should stop doing this and keep only actual errors in the array.
We can continue to log the reason normally at the debug level. Also, this information can still be inferred by the other pieces of information available (e.g. webhook settings, real errors in the array).
Currently if a webhook gives up on an item for whatever reason (e.g. channel TTL, webhook TTL, max attempts reached) a message is added to the error array stored in Zookeeper for that webhook. I am thinking we should stop doing this and keep only actual errors in the array.
We can continue to log the reason normally at the debug level. Also, this information can still be inferred by the other pieces of information available (e.g. webhook settings, real errors in the array).