-
Notifications
You must be signed in to change notification settings - Fork 572
fix(litellm): fix gen_ai.request.messages to be as expected
#5255
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
fix(litellm): fix gen_ai.request.messages to be as expected
#5255
Conversation
…inal messages and handle data URLs correctly
Semver Impact of This PR🟢 Patch (bug fixes) 📋 Changelog PreviewThis is how your changes will appear in the changelog. New Features ✨
Bug Fixes 🐛Litellm
Other
Documentation 📚
Internal Changes 🔧
🤖 This preview updates automatically when you update the PR. |
| if url.startswith("data:") and ";base64," in url: | ||
| parts = url.split(";base64,", 1) | ||
| # Remove "data:" prefix (5 chars) to get proper MIME type | ||
| mime_type = parts[0][5:] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm worried about inconsistencies in the URI decoding across different AI integrations. Can this be centralized?
Issues
Closes https://linear.app/getsentry/issue/TET-1635/redact-images-litellm