-
Notifications
You must be signed in to change notification settings - Fork 208
Sanitize OpenAI Responses API input to remove base64 encoded images #2510
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: main
Are you sure you want to change the base?
Sanitize OpenAI Responses API input to remove base64 encoded images #2510
Conversation
|
@Tehnix is attempting to deploy a commit to the PostHog Team on Vercel. A member of the Team first needs to authorize it. |
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.
1 file reviewed, 6 comments
|
This PR hasn't seen activity in a week! Should it be merged, closed, or further worked on? If you want to keep it open, post a comment or remove the |
|
Would love to see this one merged |
|
@posthog-bot This is still very much relevant, I'm running a locally patched version of @posthog/ai currently that adds this. |
|
This PR hasn't seen activity in a week! Should it be merged, closed, or further worked on? If you want to keep it open, post a comment or remove the |
|
@posthog-bot This is still very much relevant, I'm running a locally patched version of @posthog/ai currently that adds this. I'll update the PR to resolve the conflicts tomorrow though |
|
@PostHog/team-llm-analytics are you the right team to loop in here? Apologize if I shouldn't tag anyone directly, if that's the case lemme know and I'll just keep the PRs open and let your internal triage handle when you have time for it :) |
Problem
It seems only some of the API calls had added the sanitazion method for removing base64 encoded images from the input sent to PostHog (see #2134).
This would currently result in the payload sent to PostHog being too large, giving an error like this:
Most of the work was already done in #2217, it was just missing the responses API, which this PR adds.
Changes
sanitizeOpenAIRelease info Sub-libraries affected
Libraries affected
Checklist
If releasing new changes
pnpm changesetto generate a changeset file