apollo_node: add new CLI arg to parse config#14606
Conversation
PR SummaryMedium Risk Overview Native loading requires exactly two config paths (base + secrets).
Reviewed by Cursor Bugbot for commit 135115a. Bugbot is set up for automated code reviews on this repo. Configure here. |
43ac4b5 to
eadf272
Compare
19f89f8 to
7a52c90
Compare
7a52c90 to
df3b806
Compare
eadf272 to
5b34cc3
Compare
df3b806 to
5ef42b9
Compare
5b34cc3 to
135115a
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 135115a. Configure here.
| serde_json::from_reader(File::open(secret_config_path)?)?; | ||
| for (param_path, value) in secret_config { | ||
| set_nested_value(&mut nested_map, ¶m_path, value); | ||
| } |
There was a problem hiding this comment.
Native mode rejects one config file
High Severity
load_native accepts only exactly two --config_file paths via slice destructuring, yet ConfigFormat::Native, the CLI help, and test_native_config_single_file treat a lone base file as valid native loading without a secret file.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 135115a. Configure here.



No description provided.