Skip to content

fix: enable recursive search, fix openssl download URL and resolve windows path quoting errors - #1

Open
l1n00 wants to merge 1 commit into
krishnamallam:mainfrom
l1n00:fix-recursive-search-and-openssl-path
Open

l1n00 wants to merge 1 commit into
krishnamallam:mainfrom
l1n00:fix-recursive-search-and-openssl-path

Conversation

@l1n00

@l1n00 l1n00 commented Jul 22, 2026

Copy link
Copy Markdown

This PR resolves critical bugs and adds a new feature:

  1. Recursive Subfolder Search: Replaced os.listdir with Path.rglob to recursively discover all .p7m files inside the selected directory and its subfolders.
  2. Fixed Broken OpenSSL Download URL (404 Error): The previous hardcoded URL for the OpenSSL installer (v3.3.2) was returning a 404 error. Updated the installer to download Win64OpenSSL_Light-3_5_7.exe (a stable, lightweight version).
  3. Resolved "Path Not Found" Errors on Windows: Invocations of subprocess.run with shell=True and double-quoted command strings caused parsing errors on Windows when spaces were present in file/directory paths. By moving to a structured list of arguments (with shell=False), these quoting issues are completely resolved.
  4. Robust UAC Elevation Handler: Added a ctypes-based Windows UAC elevation handler to run the installer, prompting the user for permissions if running without administrative rights.

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