Staging - 12/17/2025 #18105
Staging - 12/17/2025 #18105
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
) extract model id from vertex ai passthrough routes that follow the pattern: /vertex_ai/*/models/{model_id}:* the model extraction now handles vertex ai routes by regex matching the model segment from the url path, which allows proper model identification for authentication and authorization in proxy pass-through endpoints. adds comprehensive test coverage for vertex ai model extraction including: - various vertex api versions (v1, v1beta1) - different locations (us-central1, asia-southeast1) - model names with special suffixes (gemini-1.5-pro, gemini-2.0-flash) - precedence verification (request body model over url) - non-vertex route isolation
|
|
…ng) and introduce the option to send all chunked responses in stream towards the guardrail. (#18085)
…rovider_chat_config (#18086) Claude models on Azure AI were incorrectly using AzureAIStudioConfig, causing tool calls to fail with invalid_request_error because tools remained in OpenAI format instead of being transformed to Anthropic format.
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| "vertex_location": "global", | ||
| }, | ||
| ) | ||
| assert "aiplatform.googleapis.com" in url |
Check failure
Code scanning / CodeQL
Incomplete URL substring sanitization High test
Show autofix suggestion
Hide autofix suggestion
Copilot Autofix
AI about 23 hours ago
To fix the problem, substring checks for a host/domain in a URL should be replaced by parsing the URL with a well-known library and checking its hostname explicitly. For Python, urllib.parse.urlparse can be used to extract the hostname attribute from the URL, after which the test should assert that hostname is the expected domain (aiplatform.googleapis.com). For the test on line 166, replace assert "aiplatform.googleapis.com" in url with parsing the URL and asserting on its hostname.
This will require importing urlparse from urllib.parse, if not already present in the file.
Only the assertion code in the affected test function needs to be changed; other environment checks and logic are unaffected.
| @@ -7,7 +7,7 @@ | ||
|
|
||
| import httpx | ||
| import pytest | ||
|
|
||
| from urllib.parse import urlparse | ||
| from litellm.llms.vertex_ai.image_edit.vertex_gemini_transformation import ( | ||
| VertexAIGeminiImageEditConfig, | ||
| ) | ||
| @@ -163,7 +163,8 @@ | ||
| "vertex_location": "global", | ||
| }, | ||
| ) | ||
| assert "aiplatform.googleapis.com" in url | ||
| hostname = urlparse(url).hostname | ||
| assert hostname == "aiplatform.googleapis.com" | ||
| assert "global-aiplatform.googleapis.com" not in url | ||
| assert "/locations/global/" in url | ||
|
|
9 participants
…
fix(docker): add libsndfile to Alpine image for audio processing (#18092)
Relevant issues
Pre-Submission checklist
Please complete all items before asking a LiteLLM maintainer to review your PR
tests/litellm/directory, Adding at least 1 test is a hard requirement - see detailsmake test-unitCI (LiteLLM team)
Branch creation CI run
Link:
CI run for the last commit
Link:
Merge / cherry-pick CI run
Links:
Type
🆕 New Feature
🐛 Bug Fix
🧹 Refactoring
📖 Documentation
🚄 Infrastructure
✅ Test
Changes