mirror of
https://github.com/JakeStanger/mpd-discord-rpc.git
synced 2026-07-11 17:15:28 +01:00
[GH-ISSUE #10] Crashes if connection to Discord is lost #273
Labels
No labels
bug
enhancement
help wanted
package
pull-request
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
JakeStanger/mpd-discord-rpc#273
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @JakeStanger on GitHub (Oct 15, 2020).
Original GitHub issue: https://github.com/JakeStanger/mpd-discord-rpc/issues/10
If Discord restarts or the connection to its RPC server drops for whatever reason, the program crashes.
@emersonrp commented on GitHub (Aug 15, 2023):
My Discord crashes every time I sleep/wake the computer, and occasionally for no good reason whatsoever, so I run into this a lot.
@JakeStanger commented on GitHub (Aug 16, 2023):
It looks like the underlying
discord-presencecrate doesn't surface the connection errors very well, so I'll open an issue there.EDIT: Issue here: https://github.com/jewlexx/discord-presence/issues/62
@JakeStanger commented on GitHub (Aug 21, 2023):
The above has been fixed and the library updated, so this is now resolved on
masterhere.I have noticed however that the connection retry logic in the library currently has no chill and really spams the socket/log, so I'm waiting for https://github.com/jewlexx/discord-presence/pull/73 to be merged before getting a release out or closing this.
@emersonrp commented on GitHub (Aug 22, 2023):
I've managed to fix my Discord crashing on sleep/wake, which is itself a huge win, but yeah at boot time I think the systemd service starts before Discord so needs to be restarted every boot. All that to say that this is less of a worry than it used to be but still happens, and I'm super pleased to hear there are fixes in the works. Thanks!
@ashprice commented on GitHub (Sep 1, 2023):
I do not get crashes, and I can restart discord fine, but the mentioned logging issue is causing a rather high rate of I/O on an SSD with BTRFS.
My journalctl -b log file for the last 6 hours - of a mostly idle machine - is 1.1GB in size, 5,383,274 lines long, with 5,375,627 occurrences of the words 'discord' and 'rpc.' I looked into it because I happened to be investigating I/O for other reasons. I'm not quite sure this even with the write amplification of BTRFS this can explain an unaccounted for 12.6GB+ of writes, but I suppose I will test presently...
@JakeStanger commented on GitHub (Sep 1, 2023):
It will retry and log as fast as your hardware can handle at the moment, so it will add up very quickly.
Supposedly everything is in place in the discord library, so I'm hoping to get time this weekend to have another look and get it sorted.
@JakeStanger commented on GitHub (Sep 10, 2023):
This is now fully resolved in the latest master.
I'm waiting on
discord-presenceto get a release out before I do the same, as this relies on the package's master branch currently.