JetBrains-Discord-Integration
JetBrains-Discord-Integration copied to clipboard
Fix rich presence timing issue
I had some investigation, even this plugin tries restart every 20 seconds, discord-rpc reconnection time is 60 seconds. Therfore, this is a minimal changes to make discord-rpc properly work.
Fixes #223
will this ever be merged?
Sorry, forgot about this when working on the last release. That being said I don't really see the reason for this change.
discord-rpc reconnection time is 60 seconds
This isn't really true at all, it says it right above the linked line. discord-rpc starts at a delay of 500ms and goes up to 60 seconds when repeatedly trying to reconnect, it doesn't just start at 60 seconds. Ideally the plugin would use an exponential backoff as well, but I don't see how just changing the delay to 60 seconds solves anything
@Almighty-Alpaca The rpc reconn time seems reset when try reconnect earlier. If I start the plugin without Discord, then the rpc reconnection time will gets longer than the plugin reconn time, after that we can’t never get connection using current 20 seconds. Then, how about get 1 to 120 seconds exponential backoff instead?
@Almighty-Alpaca ^^