refactor: changed user-agent, more logging, removed infinite loop

[Jaskowicz1]

This PR changes the user-agent in wsclient (to match versions.h, sorta), adds more logging in sslclient.cpp, and removes an infinite loop in discord_voice_client::thread_run.

The logging in sslclient.cpp will now send a warning when we encounter a HTTP code that isn't 204, 101, or 200. This means people can see what's going on and report it, without modifying D++ which is very helpful the more we move towards getting people to use pre-compiled binaries (view the image above the checklist for a preview).

The infinite loop in discord_voice_client::thread_run was caused by the error code 400 (you can view #714 for more info). Whilst we haven't fix the issue that #714 is about, we now tell users what's going on if things start to break (as mentioned above). The infinite loop simply checks how many times the do while loop has ran, if it's ran 5 times (without a 3 second break inbetween a loop) then we break the loop. This will mean we also don't spam the console too much with the above addition.

Code change checklist

• I have ensured that all methods and functions are fully documented using doxygen style comments.
• My code follows the coding style guide.
• I tested that my change works before raising the PR.
• I have ensured that I did not break any existing API calls.
• I have not built my pull request using AI, a static analysis tool or similar without any human oversight.

[netlify]

✅ Deploy Preview for dpp-dev ready!

Name	Link
🔨 Latest commit	ac5b21c
🔍 Latest deploy log	https://app.netlify.com/sites/dpp-dev/deploys/65113de1648c6900080aef36
😎 Deploy Preview	https://deploy-preview-883--dpp-dev.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

resolved