|
||||||||
|
This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira |
||||||||
|
||||||||
|
This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira |
||||||||
There are 2 issues with the code (I have tested with WELD-Servlet 2.1.0.Final):
First it should use "req.getServletPath();" instead of request.getRequestURI().substring(request.getContextPath().length()), in some cases, the URL contains ";jsessionID=..." it should ignore that part.
Second issue is that for each request coming in, it will do the regexp 4 times (Context Listener & Conversation Filter both before and after), and everytime it create the matcher, actually I was thinking to let the app check the url first and determine it needs to activate the context, then it will set a request attribute and later on for the same request, if the attribute is on, the WELD will activate the context. this way it will be easier and faster. (similar to the nocid option in the jsf phase listener).