tkmind/tkmind_go
1
0
Fork 0
Code Issues Pull Requests Actions 8 Packages Projects Releases Wiki Activity

fix : Custom providers with empty API keys show as configured in desktop (#6105)

Browse Source 
Signed-off-by: Abhijay007 <Abhijay007j@gmail.com>
This commit is contained in:
Abhijay Jain
2025-12-17 01:48:54 +05:30
committed by GitHub
parent 3e37f9c512
commit 9dc9f208f3
1 changed files with 1 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
crates/goose-server/src/routes/utils.rs
+1 -3
View File
@@ -94,13 +94,11 @@ pub fn inspect_keys(
pub fn check_provider_configured(metadata: &ProviderMetadata, provider_type: ProviderType) -> bool { pub fn check_provider_configured(metadata: &ProviderMetadata, provider_type: ProviderType) -> bool {
let config = Config::global(); let config = Config::global();
// TODO(Douwe): if the provider doesn't need an API key, it should be considered configured always
if provider_type == ProviderType::Custom || provider_type == ProviderType::Declarative { if provider_type == ProviderType::Custom || provider_type == ProviderType::Declarative {
if let Ok(loaded_provider) = load_provider(metadata.name.as_str()) { if let Ok(loaded_provider) = load_provider(metadata.name.as_str()) {
return config return config
.get_secret::<String>(&loaded_provider.config.api_key_env) .get_secret::<String>(&loaded_provider.config.api_key_env)
.map(|s| !s.is_empty()) .is_ok();
.unwrap_or(false);
} }
} }
// Special case: Zero-config providers (no config keys) // Special case: Zero-config providers (no config keys)