Skip to content

Enhance error handling in PeerPortchecker and TorManager; update Open…#2

Merged
MudDev merged 1 commit into
mainfrom
tor_test
Oct 21, 2025
Merged

Enhance error handling in PeerPortchecker and TorManager; update Open…#2
MudDev merged 1 commit into
mainfrom
tor_test

Conversation

@MudDev
Copy link
Copy Markdown

@MudDev MudDev commented Oct 21, 2025

This pull request improves error handling and bundled path detection for several platform-dependent features, especially when running as a PyInstaller executable on Windows. The most important changes include robust extraction of data from external service responses, improved handling of bundled binaries, and better authentication fallback logic.

Error handling improvements for external service responses:

  • Added explicit checks for missing regex matches in PeerPortchecker.py methods (checkCanyouseeme, checkIpfingerprints, checkMyaddr, checkIpv6scanner), raising exceptions with clear messages when expected data is not found. This prevents crashes due to unexpected response formats and makes debugging easier. [1] [2] [3] [4]

Bundled path detection and platform support:

  • Updated TorManager.py to detect if running as a PyInstaller executable and correctly set the path for tor.exe and meek-client.exe, improving compatibility for bundled Windows builds.
  • Enhanced OpensslFindPatch.py to check for OpenSSL DLLs in the PyInstaller _MEIPASS directory as well as the current working directory, ensuring the application finds required libraries regardless of installation method.

Authentication fallback logic:

  • Improved Tor controller authentication in TorManager.py by catching file access errors for the cookie file and attempting empty authentication if the cookie is unavailable. This increases robustness when running with external Tor instances or on different platforms.

@MudDev MudDev merged commit 1353ef5 into main Oct 21, 2025
0 of 2 checks passed
@MudDev MudDev deleted the tor_test branch October 21, 2025 22:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant