[
https://issues.apache.org/jira/browse/MYFACES-2978?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jakob Korherr resolved MYFACES-2978.
------------------------------------
Resolution: Fixed
Fix Version/s: 2.0.3-SNAPSHOT
> DefaultFaceletFactory fails to create a facelet if the facelet-path contains
> regex control characters
> -----------------------------------------------------------------------------------------------------
>
> Key: MYFACES-2978
> URL: https://issues.apache.org/jira/browse/MYFACES-2978
> Project: MyFaces Core
> Issue Type: Bug
> Components: JSR-314
> Affects Versions: 2.0.2
> Environment: cargo-maven2-plugin and jetty 6
> Reporter: Jakob Korherr
> Assignee: Jakob Korherr
> Fix For: 2.0.3-SNAPSHOT
>
>
> Using the cargo-maven2-plugin and jetty 6, you will get an application path
> like this:
> /private/var/folders/w4/w4XHIDXbF9OIP9d9vn2QjE+++TI/-Tmp-/Jetty_0_0_0_0_8080_myfaces-extcdi-cargo-test-0.9.1-SNAPSHOT.war__myfaces-extcdi-cargo-test__-lvwnpd/webapp/
> Unfortunately this path contains some regex control characters like '+' and
> thus DefaultFaceletFactory fails with the following Exception when calling
> String.replaceFirst():
> javax.faces.FacesException: java.util.regex.PatternSyntaxException: Dangling
> meta character '+' near index 48
> /private/var/folders/w4/w4XHIDXbF9OIP9d9vn2QjE+++TI/-Tmp-/Jetty_0_0_0_0_8080_myfaces-extcdi-cargo-test-0.9.1-SNAPSHOT.war__myfaces-extcdi-cargo-test__-lvwnpd/webapp/
> ^
> at
> org.apache.myfaces.shared_impl.context.ExceptionHandlerImpl.wrap(ExceptionHandlerImpl.java:241)
> at
> org.apache.myfaces.shared_impl.context.ExceptionHandlerImpl.handle(ExceptionHandlerImpl.java:156)
> at
> org.apache.myfaces.lifecycle.LifecycleImpl.executePhase(LifecycleImpl.java:191)
> at
> org.apache.myfaces.lifecycle.LifecycleImpl.execute(LifecycleImpl.java:118)
> at
> org.apache.myfaces.extensions.cdi.jsf2.impl.listener.phase.CodiLifecycleWrapper.execute(CodiLifecycleWrapper.java:55)
> at javax.faces.webapp.FacesServlet.service(FacesServlet.java:189)
> at
> org.mortbay.jetty.servlet.ServletHolder.handle(ServletHolder.java:491)
> at
> org.mortbay.jetty.servlet.ServletHandler.handle(ServletHandler.java:367)
> at
> org.mortbay.jetty.security.SecurityHandler.handle(SecurityHandler.java:185)
> at
> org.mortbay.jetty.servlet.SessionHandler.handle(SessionHandler.java:181)
> at
> org.mortbay.jetty.handler.ContextHandler.handle(ContextHandler.java:689)
> at org.mortbay.jetty.webapp.WebAppContext.handle(WebAppContext.java:391)
> at
> org.mortbay.jetty.handler.ContextHandlerCollection.handle(ContextHandlerCollection.java:146)
> at
> org.mortbay.jetty.handler.HandlerCollection.handle(HandlerCollection.java:114)
> at
> org.mortbay.jetty.handler.HandlerWrapper.handle(HandlerWrapper.java:139)
> at org.mortbay.jetty.Server.handle(Server.java:285)
> at
> org.mortbay.jetty.HttpConnection.handleRequest(HttpConnection.java:457)
> at
> org.mortbay.jetty.HttpConnection$RequestHandler.headerComplete(HttpConnection.java:751)
> at org.mortbay.jetty.HttpParser.parseNext(HttpParser.java:500)
> at org.mortbay.jetty.HttpParser.parseAvailable(HttpParser.java:209)
> at org.mortbay.jetty.HttpConnection.handle(HttpConnection.java:357)
> at
> org.mortbay.io.nio.SelectChannelEndPoint.run(SelectChannelEndPoint.java:329)
> at
> org.mortbay.thread.BoundedThreadPool$PoolThread.run(BoundedThreadPool.java:475)
> Caused by: java.util.regex.PatternSyntaxException: Dangling meta character
> '+' near index 48
> /private/var/folders/w4/w4XHIDXbF9OIP9d9vn2QjE+++TI/-Tmp-/Jetty_0_0_0_0_8080_myfaces-extcdi-cargo-test-0.9.1-SNAPSHOT.war__myfaces-extcdi-cargo-test__-lvwnpd/webapp/
> ^
> at java.util.regex.Pattern.error(Pattern.java:1713)
> at java.util.regex.Pattern.sequence(Pattern.java:1878)
> at java.util.regex.Pattern.expr(Pattern.java:1752)
> at java.util.regex.Pattern.compile(Pattern.java:1460)
> at java.util.regex.Pattern.<init>(Pattern.java:1133)
> at java.util.regex.Pattern.compile(Pattern.java:823)
> at java.lang.String.replaceFirst(String.java:2146)
> at
> org.apache.myfaces.view.facelets.impl.DefaultFaceletFactory._createViewMetadataFacelet(DefaultFaceletFactory.java:308)
> at
> org.apache.myfaces.view.facelets.impl.DefaultFaceletFactory.getViewMetadataFacelet(DefaultFaceletFactory.java:394)
> at
> org.apache.myfaces.view.facelets.impl.DefaultFaceletFactory.getViewMetadataFacelet(DefaultFaceletFactory.java:376)
> at
> org.apache.myfaces.view.facelets.FaceletViewDeclarationLanguage._getViewMetadataFacelet(FaceletViewDeclarationLanguage.java:1902)
> at
> org.apache.myfaces.view.facelets.FaceletViewDeclarationLanguage.access$000(FaceletViewDeclarationLanguage.java:128)
> at
> org.apache.myfaces.view.facelets.FaceletViewDeclarationLanguage$FaceletViewMetadata.createMetadataView(FaceletViewDeclarationLanguage.java:2136)
> at
> org.apache.myfaces.lifecycle.RestoreViewExecutor.execute(RestoreViewExecutor.java:161)
> at
> org.apache.myfaces.lifecycle.LifecycleImpl.executePhase(LifecycleImpl.java:171)
> ... 20 more
> String.replaceFirst() is used three times, in _createFacelet(),
> _createViewMetadataFacelet() and _createCompositeComponentMetadataFacelet().
> Each time it is used to remove the path of the facelet-file.
> The solution is to treat the search parameter as literal text and not as a
> regex. Thus doing exactly what String.replaceFirst() internally does,
> however, with Pattern.LITERAL set.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
