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

fix: Allow port 443 to be used with https (ollama provider) (#1052)

Browse Source
This commit is contained in:
Jeremy Collins
2025-02-03 20:43:45 -05:00
committed by GitHub
parent a515d928ab
commit c9c03222d1
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
crates/goose/src/providers/ollama.rs
+2 -1
View File
@@ -65,7 +65,8 @@ impl OllamaProvider {
.map_err(|e| ProviderError::RequestFailed(format!("Invalid base URL: {e}")))?; .map_err(|e| ProviderError::RequestFailed(format!("Invalid base URL: {e}")))?;
// Set the default port if missing // Set the default port if missing
if base_url.port().is_none() { let explicit_default_port = self.host.ends_with(":80") || self.host.ends_with(":443");
if base_url.port().is_none() && !explicit_default_port {
base_url.set_port(Some(OLLAMA_DEFAULT_PORT)).map_err(|_| { base_url.set_port(Some(OLLAMA_DEFAULT_PORT)).map_err(|_| {
ProviderError::RequestFailed("Failed to set default port".to_string()) ProviderError::RequestFailed("Failed to set default port".to_string())
})?; })?;