-
Notifications
You must be signed in to change notification settings - Fork 2.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
docs: attributes undocumented for certain logging plugins #9190
Comments
@Gallardot I'm taking this up! |
@Neilblaze Are you still working on this issue? |
@navendu-pottekkat Sorry for the delay. I'm working on another issue, so if anyone wants to pick this up, please feel free to assign :) |
@kayx23 can assignees it to me. I'll do it |
@smileby Thank you. Assigned. |
Going through some logging plugins now and noticed these options are not added to all logging plugins, such as |
Current State
Thanks to APISIX's excellent design, I found that many logging plug-ins support features such as
include_req_body
,include_req_body_expr
,include_resp_body
,include_resp_body_expr
, such asfile-logger
,http-logger
,loggly
,tencent-cloud-cls
. However, this feature does not appear in the plug-in documentation or in the plug-in schema definition.Desired State
Maybe we should add the above.
The text was updated successfully, but these errors were encountered: