-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Add size limit for content of comment on action ui #12881
Conversation
Signed-off-by: a1012112796 <1012112796@qq.com>
Codecov Report
@@ Coverage Diff @@
## master #12881 +/- ##
==========================================
- Coverage 43.10% 43.10% -0.01%
==========================================
Files 656 656
Lines 72348 72353 +5
==========================================
- Hits 31189 31186 -3
- Misses 36108 36113 +5
- Partials 5051 5054 +3
Continue to review full report at Codecov.
|
Co-authored-by: mrsdizzie <info@mrsdizzie.com>
@a1012112796 Please send backport to v1.12 |
Signed-off-by: a1012112796 <1012112796@qq.com> Co-authored-by: mrsdizzie <info@mrsdizzie.com> Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
Signed-off-by: a1012112796 <1012112796@qq.com> Co-authored-by: mrsdizzie <info@mrsdizzie.com> Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com> Co-authored-by: mrsdizzie <info@mrsdizzie.com> Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com> Co-authored-by: techknowlogick <techknowlogick@gitea.io>
Previous PR go-gitea#12881 causes out of bounds panic by working on wrong string.
This code and backport causes panic for large comments -- will send PR to fix |
Previous PR #12881 causes out of bounds panic by working on wrong string.
Previous PR go-gitea#12881 causes out of bounds panic by working on wrong string.
As tile, If the content of comment is to large, just show 200 words on action ui.
fix #12870