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

update:split #710

Merged
merged 4 commits into from
Sep 20, 2024
Merged

update:split #710

merged 4 commits into from
Sep 20, 2024

Conversation

SAKURA-CAT
Copy link
Contributor

Description

为了减少后端压力以及解决日志文本(进度条)过长导致服务器被冲的问题,添加上传的日志和媒体文件的长度限制,分别为一行200字符和一次108媒体(列表长度)

@SAKURA-CAT SAKURA-CAT added the 🐛 bug Something isn't working label Sep 20, 2024
@SAKURA-CAT SAKURA-CAT self-assigned this Sep 20, 2024
@Zeyi-Lin
Copy link
Member

Zeyi-Lin commented Sep 20, 2024

关于进度条问题,更本质的解法还是对/r做优化

@SAKURA-CAT
Copy link
Contributor Author

SAKURA-CAT commented Sep 20, 2024

关于进度条问题,更本质的解法还是对/r做优化

我知道,上述的限制主要是避免长文本/长日志问题,例如用户手动的 print("x" * 极大数)
进度条问题只能说是一种用户无意识的上传长文本

@SAKURA-CAT SAKURA-CAT merged commit df72eb5 into main Sep 20, 2024
5 checks passed
@SAKURA-CAT SAKURA-CAT deleted the feat/limit-len branch September 20, 2024 06:38
@SAKURA-CAT
Copy link
Contributor Author

关于进度条适配请跟踪 #421

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants