Harden Config Parsing / Validation - #268
Open
benthecarman wants to merge 4 commits into
Open
Conversation
|
👋 I see @wpaulino was un-assigned. |
Stop API key resolution when a loaded configuration names an unsupported network. This prevents a malformed config from selecting the Bitcoin key. This commit was created with assistance from Codex.
Stop API key lookup after an explicit storage directory is selected. A missing instance key must not fall back to another node's credentials. This commit was created with assistance from Codex.
Return an error when the operator selects a config path that is missing or is not a file. This prevents silent fallback to the default node. This commit was created with assistance from Codex.
Return an error when an explicit MCP config path is missing or is not a file. This prevents the bridge from selecting the default node instead. This commit was created with assistance from Codex.
benthecarman
force-pushed
the
codex/loupe-client-config
branch
from
September 1, 2026 02:45
a56ef0c to
36cc238
Compare
benthecarman
requested review from
joostjager
and removed request for
wpaulino
September 1, 2026 16:35
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Various issues by project loupe where we could silently fallback to a default config or file location in an unexpected way. Better to force the user to be explicit upfront to prevent potential issues.