Github user SolidWallOfCode commented on a diff in the pull request:

    https://github.com/apache/trafficserver/pull/1210#discussion_r87404997
  
    --- Diff: plugins/experimental/log_requests/README ---
    @@ -0,0 +1,37 @@
    +log_requests
    +==============
    +
    +This plugin is used to help debug errored out transactions. 
    +We define errored out as any transaction returning a 4xx/5xx 
    +response code. By default, all transactions that return a 
    +4xx/5xx response code to the client will get its 
    +request/response headers logged into error.log.
    +
    +We also provide a blacklist flag via the plugin arguments. Take 
    +for example this config in plugin.config:
    +
    +    log_requests.so --no-log 414 502
    +
    +Then log_requests would log every 4xx/5xx response code transaction
    +EXCEPT for 414 and 502.
    +
    +
    +Usage
    +------
    +When compiling ATS, run configure with the following flags:
    +
    +    ./configure --enable-experimental-plugins
    +
    +
    +Then in plugin.config:
    +    
    +    log_requests.so [--log-proxy] [--no-log [error code 1[,error code 
2[,...]]]]
    +
    +    In other words, the --log-proxy and --no-log flags are optional. 
    +    --no-log should be followed with a comma separated list of error
    +    codes to blacklist.
    +
    +    Note: There is an interesting bug where if you want to enable 
    +    --log-proxy, it has to come before the --no-log flag. I'm not
    +    sure why this happens, but you'll have to just live with it
    +    until I figure it out.
    --- End diff --
    
    See previous comment ^



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to