[Infra] CI/CD Fixes - Nightly Release Feb 12 - #21081
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
Greptile OverviewGreptile SummaryCherry-picks CI/CD and bug fixes from the
Confidence Score: 4/5
|
| Filename | Overview |
|---|---|
| litellm/completion_extras/litellm_responses_transformation/transformation.py | Refactors transform_request into smaller helper methods. Introduces silent truncation of user param > 64 chars (previously silently dropped). Behavioral change in how overlong user strings are handled. |
| litellm/integrations/cloudzero/transform.py | Bug fix: resource/id field now correctly uses resource_id (CZRN) instead of model name. |
| litellm/litellm_core_utils/exception_mapping_utils.py | Adds exclusions for OpenAI user param validation errors (string_above_max_length, Invalid 'user') so they are not misclassified as context window errors. |
| litellm/llms/custom_httpx/aiohttp_transport.py | Refactors aiohttp request call to build kwargs dict first, conditionally adding ssl only when non-None. Avoids passing None as ssl parameter. |
| litellm/proxy/_experimental/mcp_server/auth/user_api_key_auth_mcp.py | Adds fallback DB lookup (with caching via user_api_key_cache) for key_object_permission when not pre-loaded. Uses inline imports from proxy_server. |
| litellm/proxy/spend_tracking/spend_management_endpoints.py | Fixes _build_ui_spend_logs_response to handle row as either dict or model object, preventing AttributeError when row is already a dict. |
| requirements.txt | Security updates: Pillow 11.0.0 → 12.1.1 (GHSA-cfh3-3jmp-rvhc), cryptography 44.0.1 → 46.0.5 (GHSA-r6ph-v2qm-q3c2). Moved to top of file with CVE annotations. |
Sequence Diagram
sequenceDiagram
participant Client
participant TransformRequest as transform_request()
participant MapParams as _map_optional_params
participant BuildLitellm as _build_sanitized_litellm_params
participant MergeRequest as _merge_responses_api_request
participant ResponsesAPI as Responses API
Client->>TransformRequest: model, messages, optional_params, litellm_params
TransformRequest->>TransformRequest: convert_chat_completion_messages_to_responses_api()
TransformRequest->>MapParams: optional_params, responses_api_request
MapParams-->>TransformRequest: (mutated responses_api_request)
TransformRequest->>BuildLitellm: litellm_params
BuildLitellm-->>TransformRequest: sanitized_litellm_params
TransformRequest->>TransformRequest: Build request_data dict
TransformRequest->>MergeRequest: request_data, responses_api_request, instructions
MergeRequest->>MergeRequest: Truncate user if > 64 chars
MergeRequest-->>TransformRequest: (mutated request_data)
TransformRequest-->>Client: request_data
Client->>ResponsesAPI: request_data
Last reviewed commit: 3f8a1cf
Comment on lines
+299
to
+304
| elif key == "user" and isinstance(value, str): | ||
| # OpenAI API requires user param to be max 64 chars - truncate if longer | ||
| if len(value) <= 64: | ||
| request_data["user"] = value | ||
| else: | ||
| request_data["user"] = value[:64] |
Contributor
There was a problem hiding this comment.
Behavioral change: previously, user values > 64 chars were silently dropped (not sent at all). Now they are silently truncated to 64 chars. This could cause different users with long IDs sharing the same 64-char prefix to become indistinguishable. Consider logging when truncation occurs so callers can detect and fix their user param.
fzowl
pushed a commit
to fzowl/litellm
that referenced
this pull request
Jun 24, 2026
…ges_feb12 [Infra] CI/CD Fixes - Nightly Release Feb 12
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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-unit@greptileaiand received a Confidence Score of at least 4/5 before requesting a maintainer reviewCI (LiteLLM team)
Branch creation CI run
Link:
CI run for the last commit
Link:
Merge / cherry-pick CI run
Links:
Type
🚄 Infrastructure
Changes
This PR cherry-picks commits from https://github.com/BerriAI/litellm/tree/litellm_release_day_02_12_2026. Some commits from this branch should not be merged with main because they are workarounds, hence I am making a new branch to merge the CI/CD fixes into main.