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

Leif Hedstrom commented on TS-943:
----------------------------------

Looks good to me. The only minor thing would be that it could be nice to unify 
the code that fetches the config value into one place, instead of two with this 
patch. I'm talking about

IOCORE_ReadConfigInteger(should_filter_int, "proxy.config.net.defer_accept");


Which is also used in the section where we support defer accept for Linux. Not 
a huge deal.

> Support for HTTP accept filter
> ------------------------------
>
>                 Key: TS-943
>                 URL: https://issues.apache.org/jira/browse/TS-943
>             Project: Traffic Server
>          Issue Type: New Feature
>          Components: Network
>    Affects Versions: 3.1.0
>            Reporter: Theo Schlossnagle
>            Assignee: Theo Schlossnagle
>            Priority: Minor
>         Attachments: accept_filter.patch
>
>   Original Estimate: 2h
>  Remaining Estimate: 2h
>
> Add support for accept filters (Solaris and FreeBSD) triggered by 
> proxy.config.net.defer_accept

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to