Skip to content

fix(extension): name the missing dependency when Windows load fails - #704

Merged
adsharma merged 2 commits into
LadybugDB:mainfrom
freya0926:fix/windows-extension-missing-dep-message
Jul 25, 2026
Merged

fix(extension): name the missing dependency when Windows load fails#704
adsharma merged 2 commits into
LadybugDB:mainfrom
freya0926:fix/windows-extension-missing-dep-message

Conversation

@freya0926

@freya0926 freya0926 commented Jul 19, 2026

Copy link
Copy Markdown
Contributor

Windows reports a missing transitive dependency as ERROR_MOD_NOT_FOUND against the library passed to LoadLibraryW, so LOAD EXTENSION fts on a machine without OpenSSL 3 fails with "the specified module could not be found" — naming the extension, which is present. Users are left guessing which module is actually meant (#685).

When the extension file itself exists, this adds a line saying the missing module is something the extension links against, and names the OpenSSL 3 DLLs.

Scope: this only improves the message. It does not fix #685@adsharma confirmed the core links OpenSSL dynamically, which means the core has already loaded libssl-3-x64.dll before any extension loads, so the fix for the underlying packaging problem belongs elsewhere.

Testing: compiles clean on MSVC 14.44 (VS 2022 Build Tools, Windows SDK 10.0.26100), full static Release build links. I could not exercise the message at runtime — lbug_shell.exe built here crashes with 0xC0000409 on database open, which I confirmed is pre-existing by reproducing it with this change reverted.

Windows raises ERROR_MOD_NOT_FOUND against the library passed to
LoadLibraryW, so a missing transitive dependency is reported as "the
specified module could not be found" naming the extension -- which is
present. Users are left guessing which module is meant.

When the extension file itself exists, say that the missing module is
something it links against and name the OpenSSL 3 DLLs, which are the
usual culprits on Windows.

Refs LadybugDB#685
@adsharma

Copy link
Copy Markdown
Contributor

Closing based on no bots policy. Humans generating AI assisted PRs is ok.

@adsharma adsharma closed this Jul 19, 2026
@freya0926

freya0926 commented Jul 24, 2026

Copy link
Copy Markdown
Contributor Author

Thanks for the clarification — I removed the "Generated with Claude Code" footer from the description above. To be clear: I'm a human, and I used Claude Code as a tool to help draft this fix, the same way one might use any other coding assistant. Happy to answer questions about the change itself if useful.

@adsharma adsharma reopened this Jul 24, 2026
@adsharma
adsharma merged commit e3aaa8a into LadybugDB:main Jul 25, 2026
8 checks passed
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.

Bug: [Windows] FTS extension 0.18.1 requires OpenSSL DLLs that are not distributed

2 participants