sstallion / clipsock Goto Github PK

When resolving the listen address, the first address returned by ParseNetworkString is expected. More often than not, this seems to resolve to an IPv6 address, which is different than behavior found in SSH clients like PuTTY and SecureCRT. This should be addressed so that localhost can be used as the default listen address rather than 127.0.0.1.

Clipboard data is currently limited to ANSI due to CF_TEXT being passed to SetClipboardData(). UTF-8 should be possible using CF_UNICODE and MultiByteToWideChar, however this causes issues with buffering. This could be solved by introducing a second buffer, but other methods should be investigated before doubling the memory footprint.

When ClipSock runs for the first time, the server will be started prior to showing the settings dialog. This was laziness on my part since this situation only occurs once per installation, but it's messy. Ideally, the server will not start until the user has confirmed the default settings.

Server status is written to the tooltip text for the taskbar notification area icon, however this can be easy to miss. It would be nice to provide a small indicator (eg. green/red) to show server status visually.

Current guidance from Microsoft with respect to using merge modules in .msi installers for redistributable packages:

If you deploy your code only as an .msi installer, we recommend you don't include the merge modules in the installer. Instead, report a failure to the user when the required Redistributable libraries aren't found. In the failure dialog, include instructions on how to find and install the latest Redistributable package. For more information and links to Redistributable packages, see The latest supported Visual C++ downloads.

Given the above, WiX should updated to show an error dialog if runtime libraries cannot be found. As a stop-gap, this limitation should be documented in README.md.