search

ForkGG / Fork-legacy

Fork: Minecraft Server Manager GUI for Windows
https://www.fork.gg
MIT License
102 stars 15 forks source link

Need to error trap "Failed connection to discord-ipc-(X)" otherwise it bloats consoleLog.txt #30

Closed The-Monkey-King closed 6 months ago

The-Monkey-King commented 2 years ago

Issue: failing a connection to discord fills the consoleLog.txt while Fork is running.

Suggestions:

  1. Application should only attempt to connect to discord-ipx-(0-9) three times. (30 attempts to reach any discord-ipx)
  2. Application connection attempts to be recorded in consoleLog.txt as "attempting to connect to discord-ipx.. (1 of 3)" ex cetera.
  3. When the application fails to connect to discord-ipx-(0-9) (times out), after three tries, the message should be placed in the error log, "Failed to reach discord IPX."

consoleLog.txt

ChristianKernDev commented 6 months ago

This can not be trapped, as it is logged by the DiscordRPC package. This is intended and won't be fixed: https://github.com/Lachee/discord-rpc-csharp/issues/142

Disabled logging for DiscordRPC errors as a fix