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

block · Feb 4, 2025 · c9c0322 · c9c0322
1 parent a515d92
commit c9c0322
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/crates/goose/src/providers/ollama.rs b/crates/goose/src/providers/ollama.rs
@@ -65,7 +65,8 @@ impl OllamaProvider {
             .map_err(|e| ProviderError::RequestFailed(format!("Invalid base URL: {e}")))?;
 
         // 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(|_| {
                 ProviderError::RequestFailed("Failed to set default port".to_string())
             })?;