Skip to content

docs: update rocketmq-logger.md, add max req/resp body size attributes description(3.10-) #12173

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

Open
wants to merge 1 commit into
base: release/3.12
Choose a base branch
from

Conversation

yuanrui01
Copy link

This PR updates the documentation to include descriptions for two new configuration attributes introduced in version 3.10: max_req_body_bytes and max_resp_body_bytes. These attributes allow users to limit the recorded size of request and response bodies when include_req_body or include_resp_body is enabled, respectively, with a default value of 524288 bytes. Bodies exceeding these limits are truncated, enhancing memory efficiency and system stability.

@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. doc Documentation things labels Apr 22, 2025
@yuanrui01 yuanrui01 changed the title Update rocketmq-logger.md, add max req/resp body size attributes description(3.10-) docs: update rocketmq-logger.md, add max req/resp body size attributes description(3.10-) Apr 23, 2025
@Baoyuantop
Copy link
Contributor

I didn't find these two attributes in the code. 🤔

@yuanrui01
Copy link
Author

I didn't find these two attributes in the code. 🤔

I found it in log-util.lua
conf.max_req_body_bytes
conf.max_resp_body_bytes
Maybe this is implemented as a public property of all logging plugins?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Documentation things size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants