Skip to content

Resolve CLI API key and TLS cert from config storage dir#145

Merged
benthecarman merged 1 commit intolightningdevkit:mainfrom
joostjager:cli-resolve-api-key-and-cert-from-storage-dir
Mar 11, 2026
Merged

Resolve CLI API key and TLS cert from config storage dir#145
benthecarman merged 1 commit intolightningdevkit:mainfrom
joostjager:cli-resolve-api-key-and-cert-from-storage-dir

Conversation

@joostjager
Copy link
Contributor

When a custom config file is passed to ldk-server-cli, the API key and TLS certificate lookups now check the config's storage.disk.dir_path before falling back to the platform default data directory. Previously these files were only looked up in the default location, causing connection failures when using a non-default storage directory.

@ldk-reviews-bot
Copy link

ldk-reviews-bot commented Mar 11, 2026

👋 Thanks for assigning @benthecarman as a reviewer!
I'll wait for their review and will help manage the review process.
Once they submit their review, I'll check if a second reviewer would be helpful.

When a custom config file is passed to ldk-server-cli, the API key and
TLS certificate lookups now check the config's storage.disk.dir_path
before falling back to the platform default data directory. Previously
these files were only looked up in the default location, causing
connection failures when using a non-default storage directory.

AI tools were used in preparing this commit.
@joostjager joostjager force-pushed the cli-resolve-api-key-and-cert-from-storage-dir branch from fded6a6 to 1ae3314 Compare March 11, 2026 13:44
@benthecarman benthecarman self-requested a review March 11, 2026 13:45
@benthecarman benthecarman merged commit a4da3ef into lightningdevkit:main Mar 11, 2026
7 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.

3 participants