[
https://issues.apache.org/jira/browse/HTTPCORE-381?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14001639#comment-14001639
]
Oleg Kalnichevski commented on HTTPCORE-381:
--------------------------------------------
Another possibility would be using an extended interface
{code:java}
public interface HttpContextAwareAsyncRequestHandlerMapper extends
HttpAsyncRequestHandlerMapper {
HttpAsyncRequestHandler<?> lookup(HttpRequest request, HttpContext context);
}
{code}
I would still prefer not to have to resort to that if we are really talking
about saving a few CPU cycles.
Oleg
> Missing HttpContext in HttpAsyncRequestHandlerMapper
> ----------------------------------------------------
>
> Key: HTTPCORE-381
> URL: https://issues.apache.org/jira/browse/HTTPCORE-381
> Project: HttpComponents HttpCore
> Issue Type: Improvement
> Components: HttpCore NIO
> Affects Versions: 4.3.2
> Reporter: Tobias Bieniek
>
> It is a common use case to use a custom HttpAsyncRequestHandlerMapper
> implementation to match the URLs against a list of regular expressions. Often
> these regular expressions include capture groups that can be used to extract
> parameters from the URL.
> In `/user/(\d+)/profile` for example the group can be extracted and converted
> to an integer. While the HttpAsyncRequestHandlerMapper implementation already
> does the regex matching, it has no place to save the matching results since
> the HttpContext is not passed to the mapper.
> The naive way to fix this is to include the HttpContext in the lookup()
> method interface, but unfortunately that would be a breaking change. I hope
> that someone is able to come up with a better solution for this.
> This issue is to some degree related to
> https://issues.apache.org/jira/browse/HTTPCORE-308
--
This message was sent by Atlassian JIRA
(v6.2#6252)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
