kamly commented on PR #9712:
URL: https://github.com/apache/apisix/pull/9712#issuecomment-1627367448

   > Refer to 
[include_req_body](https://github.com/apache/apisix/blob/bb0a0e6e87eb0f15807be6fed0e7c983855d491c/apisix/utils/log-util.lua#L166C13-L166C29)
 and `include_req_body_expr` to see if it is better to implement an 
`include_req_header` and `include_req_header_expr` feature so that other 
logging plugins can support this feature without each plugin implementing it 
separately.
   > 
   > @kamly @monkeyDluffy6017 WDYT?
   
   good idea,  but i see file-logger not support `include_req_body_expr`
   
   @monkeyDluffy6017 how do you think ?
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to