[jira] [Comment Edited] (TS-2636) Enhance ATS custom logging to support WIPE filter action

["Yunkai Zhang (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/TS-2636?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13946133#comment-13946133 ] 

Yunkai Zhang edited comment on TS-2636 at 3/25/14 4:01 AM:
-----------------------------------------------------------

Hi [~sudheerv] [~zwoop]:

With more deep thought, inspired by Leif, I got a better solution:

Since "WIPE" action only applys URL fileds(eg. %<cquuc>), strictly speaking, it's not a *generic* filter ACTION, but a *special* filter FUNCTION that applys certain fields. So, why not just create a new function to achieve this purpose?

For example, we can add a new filed-function(somewhat like aggregated function): {{%<WIPE(field, param1, param2, ...)>}}
{code}
%<WIPE(cquuc, token)>  // WIPE "token" parameter
#<WIPE(cquuc, token, userid, ...)> // WIPE "token"/"userid"/... parameters.
{code}

With this filed-function, it's will be more efficient -- it will be only executed on certain fields.



was (Author: yunkai):
Hi [~sudheerv] [~zwoop]:

With more deep thought, inspired by Leif, I got a better solution:

Since "WIPE" action only applys URL fileds(eg. %<cquuc>), strictly speaking, it's not a *generic* filter ACTION, but a *special* filter FUNCTION that applys certain fields. So, why not just create a new function to achieve this purpose?

For example, we can add a new filed-function(somewhat like aggregated function): {{%<WIPE(filed, param1, param2, ...)>}}
{code}
%<WIPE(cquuc, token)>  // WIPE "token" parameter
#<WIPE(cquuc, token, userid, ...)> // WIPE "token"/"userid"/... parameters.
{code}

With this filed-function, it's will be more efficient -- it will be only executed on certain fields.


> Enhance ATS custom logging to support WIPE filter action
> --------------------------------------------------------
>
>                 Key: TS-2636
>                 URL: https://issues.apache.org/jira/browse/TS-2636
>             Project: Traffic Server
>          Issue Type: New Feature
>          Components: Logging
>            Reporter: Sudheer Vinukonda
>            Assignee: Yunkai Zhang
>              Labels: Review
>             Fix For: 5.0.0
>
>         Attachments: ts2636.diff
>
>
> Currently, ATS custom logging supports LogFilters with actions Accept, Reject. This feature request is to add a new filter action "WIPE". WIPE can be used in hiding sensitive parameters (e.g. userid, password) within the query part of a URL. Attached is a draft version of the patch. Kindly review and comment. 



--
This message was sent by Atlassian JIRA
(v6.2#6252)