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

Oleg Kalnichevski commented on HTTPCLIENT-1806:
-----------------------------------------------

Baosen,
That essentially means that even if I commit your code now I am very likely to 
remove it in coming weeks while refactoring the existing request processing and 
execution code. 

If your motivation is contribute to the project rather than getting this 
particular feature into the next release I am pretty sure there are plenty of 
opportunities to do so. Just let me know and we will surely figure out a way 
for you contribute something both challenging and useful.

Oleg

> Add a route inteceptor
> ----------------------
>
>                 Key: HTTPCLIENT-1806
>                 URL: https://issues.apache.org/jira/browse/HTTPCLIENT-1806
>             Project: HttpComponents HttpClient
>          Issue Type: Improvement
>          Components: HttpClient (async), HttpClient (classic)
>            Reporter: Baosen Cui
>            Priority: Minor
>             Fix For: 5.0
>
>
> I had meet a problem that I need to log the time cost by  method 
> “establishRoute”( in httpclient ).
> Since there seems nothing to support, I wrote a interceptor into HttpCore 
> coping the structure of “HttpProcessor"
> Would this be accepted? Or there has already something alike?
> @ThreadSafe // provided injected dependencies are immutable
> public final class ImmutableRouteProcessor implements RouteProcessor {
> ...
> }
> public interface PostRouteInterceptor {
>   void postProcess(final HttpRequest request, final HttpContext context);
> }
> public interface PreRouteInterceptor {
>   void preProcess(final HttpRequest request, final HttpContext context);
> }
> public interface RouteProcessor extends PreRouteInterceptor, 
> PostRouteInterceptor {
> }



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to