Skip to content
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

fix(ai-proxy): remove passthrough #12014

Merged

Conversation

shreemaan-abhishek
Copy link
Contributor

Description

During development of ai-proxy this feature went un-noticed during back and forths of code review and it was found that it doesn't work now. So we can remove it for now and maybe add it later with full functionality.

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Mar 1, 2025
nic-6443
nic-6443 previously approved these changes Mar 2, 2025
membphis
membphis previously approved these changes Mar 2, 2025
@shreemaan-abhishek shreemaan-abhishek dismissed stale reviews from membphis and nic-6443 via 009425c March 3, 2025 02:41
@shreemaan-abhishek shreemaan-abhishek merged commit 312f1e5 into apache:master Mar 3, 2025
32 of 33 checks passed
@shreemaan-abhishek shreemaan-abhishek deleted the ai-proxy-passthr branch March 3, 2025 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants