-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
enhance: cut tracing content as unicode for safety #3342
Conversation
tomorrow fix |
建议测试一下比较大的内容上报,防止卡住,以及文件上传的 |
It is recommended to test the content comparison to prevent getting stuck, as well as the file upload. |
OK,我加个大 body 测试,现在的测试失败是什么原因导致的? |
OK, I'll add a big body test. What's the reason for the current test failure? |
你可以搜索一下issues的记录,有同学反馈过,卡住的情况 |
You can search the records of issues. Some students have reported that they are stuck. |
感觉如果判断是 HTTP 文件上传是不是直接不记录更好. |
I feel that if it is judged to be an HTTP file upload, it would be better not to record it directly. |
我自己是没有碰到过这种问题,只有trace上报服务器配置错误,导致启动不起来的问题。 |
I have never encountered this kind of problem myself. Only trace reported a server configuration error, which caused the server to fail to start. |
@seth-shi If this pr is ready again, please AT me. |
it`s already ok, is there anything need to fix ? |
more screen string field contains invalid UTF-8 on gzip content
safeContent for httpRequestBody and httpResponseBody