Skip to content

Conversation

@HecarimV
Copy link
Contributor

@HecarimV HecarimV commented May 8, 2025

Ⅰ. Describe what this PR did

ai-statistics插件修改RequestBodyBufferLimit

Ⅱ. Does this pull request fix one issue?

fix #2188

@CH3CHO
Copy link
Collaborator

CH3CHO commented May 8, 2025

建议加个配置项,不然你配多大都可能会遇到比这个还大的 body。

当然终极解决方案是改成流式解析。

@HecarimV
Copy link
Contributor Author

HecarimV commented May 8, 2025

建议加个配置项,不然你配多大都可能会遇到比这个还大的 body。

当然终极解决方案是改成流式解析。

我看ai-proxy配的这么大,100 MB应该够大了吧

@CH3CHO
Copy link
Collaborator

CH3CHO commented May 8, 2025

建议加个配置项,不然你配多大都可能会遇到比这个还大的 body。
当然终极解决方案是改成流式解析。

我看ai-proxy配的这么大,100 MB应该够大了吧

好吧,那也不是不行。

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 45.73%. Comparing base (ef31e09) to head (67fb867).
Report is 495 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2192      +/-   ##
==========================================
+ Coverage   35.91%   45.73%   +9.82%     
==========================================
  Files          69       79      +10     
  Lines       11576    12895    +1319     
==========================================
+ Hits         4157     5897    +1740     
+ Misses       7104     6651     -453     
- Partials      315      347      +32     

see 77 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@johnlanni johnlanni merged commit cd271c1 into alibaba:main May 8, 2025
10 checks passed
daixijun pushed a commit to daixijun/higress that referenced this pull request May 9, 2025
VinceCui pushed a commit to VinceCui/higress that referenced this pull request May 21, 2025
ink-hz pushed a commit to ink-hz/higress-ai-capability-auth that referenced this pull request Nov 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[plugins]: ai-statistics leading to 413 error for large request payloads

4 participants