Skip to content

chore: rectify business logic/code in ai-proxy#12055

Merged
shreemaan-abhishek merged 8 commits intoapache:masterfrom
shreemaan-abhishek:cleanup-ai-proxy
Mar 17, 2025
Merged

chore: rectify business logic/code in ai-proxy#12055
shreemaan-abhishek merged 8 commits intoapache:masterfrom
shreemaan-abhishek:cleanup-ai-proxy

Conversation

@shreemaan-abhishek
Copy link
Copy Markdown
Contributor

Description

This PR does the following fixes:

checks if the response content-type header is not nil before doing string.find
return 504 if timeout error occurs
weight is required when configuring upstream nodes in ai-proxy-multi
bypass nginx upstream in ai-proxy-multi
add debug logs for ai usage
remove max timeout limit in schema
return http code when read chunk causes error

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 size:L This PR changes 100-499 lines, ignoring generated files. chore labels Mar 17, 2025
nic-6443
nic-6443 previously approved these changes Mar 17, 2025
Copy link
Copy Markdown
Member

@membphis membphis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@shreemaan-abhishek shreemaan-abhishek merged commit 99a0792 into apache:master Mar 17, 2025
30 checks passed
@shreemaan-abhishek shreemaan-abhishek deleted the cleanup-ai-proxy branch March 17, 2025 13:44
@kayx23 kayx23 mentioned this pull request Mar 24, 2025
5 tasks
laz-xyr pushed a commit to laz-xyr/apisix that referenced this pull request Jun 19, 2025
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