fix: check deny list in for certain chat models#6589
Conversation
There was a problem hiding this comment.
Code Review
This pull request integrates checkDenyList security checks across multiple chat model components to validate base URLs and paths. The review feedback highlights a critical SSRF vulnerability in the Ollama component where an empty baseUrl defaults to localhost and bypasses the check. Additionally, the reviewer suggests combining redundant if conditions for baseUrl and basePath to improve code readability.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…hatCloudflareWorkersAI.ts Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
No description provided.