sentry-unity icon indicating copy to clipboard operation
sentry-unity copied to clipboard

Add time option for log, error, warning debouncers

Open oddgames-david opened this issue 3 years ago • 3 comments

With unity it's very common for the first exception to be only one that truly needs fixing and a debounce time of 1 second doesn't really cut it, I'd use a 30 second timer so my sentry review box doesn't fill up with stuff I really don't need to see.

oddgames-david avatar Dec 24 '21 01:12 oddgames-david

The debouncing strategy is something we can improve. Just based on time is also not good enough as it often drops legit events. There has been some other issues discussing this but we should bring this topic back to improve this feature going forward

Related to #379

bruno-garcia avatar Jan 04 '22 20:01 bruno-garcia

I agree with the idea. In addition to the checkbox we could even have a slider or an inputbox for seconds/milliseconds next to it. Is this something you would be willing to contribute to the SDK with a PR? I'd be more than happy to help out/review it.

bitsandfoxes avatar Jan 10 '22 15:01 bitsandfoxes

I would also like to suggest an improvement, that logs of the same type but with different messages should not get debounced.

aaman-git avatar Mar 24 '22 15:03 aaman-git