Hi,

2015-09-28 14:26 GMT+03:00 Mark Thomas <ma...@apache.org>:
>
> The proposed Apache Tomcat 8.0.27 release is now available for voting.
>
> The main changes since 8.0.26 are:
>
> - Correctly handle \${ vs \$ escaping in JSP and EL
>
> - Fix for issues with NIO + SSL + sendfile
>
> - Various TLD parsing fixes
>
> - Fix multiple (mostly rare and/or zero impact) concurrency issues
>
>
> Note: Due to an issues with the Symantec code signing service (the
>       certificates used to access the web-based admin GUI have all
>       expired and infra is having to jump through some hoops to get
>       new ones issues) the Windows Installer package has NOT been
>       signed for this release.
>
> There is the usual collection of bug fixes, new features and
> performance improvements. For full details, see the changelog:
>
http://svn.us.apache.org/repos/asf/tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml
>
> It can be obtained from:
> https://dist.apache.org/repos/dist/dev/tomcat/tomcat-8/v8.0.27/
> The Maven staging repo is:
> https://repository.apache.org/content/repositories/orgapachetomcat-1051/
> The svn tag is:
> http://svn.apache.org/repos/asf/tomcat/tc8.0.x/tags/TOMCAT_8_0_27/
>
> The proposed 8.0.27 release is:
> [ ] Broken - do not release
> [ ] Stable - go ahead and release as 8.0.27

I have the following use case [1]. It used to be working with the previous
version (8.0.26).

Now it fails with the following [2].

I didn't find in the spec whether this is allowed or not :( however the
exception is bad.

Regards,
Violeta

[1]

Index: org/apache/el/TestELInJsp.java
===================================================================
--- org/apache/el/TestELInJsp.java (revision 1705912)
+++ org/apache/el/TestELInJsp.java (working copy)
@@ -318,6 +318,7 @@
         assertEcho(result, "20-4");
         assertEcho(result, "21-[{value=11}, {value=12}, {value=13},
{value=14}]");
         assertEcho(result, "22-[{value=11}, {value=12}, {value=13},
{value=14}]");
+        assertEcho(result, "23-\\\"foo\\\"");
     }

     @Test
Index: webapp/el-misc.jsp
===================================================================
--- webapp/el-misc.jsp (revision 1705912)
+++ webapp/el-misc.jsp (working copy)
@@ -41,4 +41,5 @@
     <p>20-${{1,2,3,4}.stream().max().orElse(-1)}</p>
     <tags:echo
echo='21-${{1,2,3,4}.stream().sorted().map(u->{"value":u+10}).toList()}' />

 <p>22-${{1,2,3,4}.stream().sorted().map(u->{"value":u+10}).toList()}</p>
 </body>
+    <tags:echo echo="23-${\"\\\"foo\\\"\"}" />
 </html>
\ No newline at end of file

[2]

 org.apache.catalina.core.StandardWrapperValve.invoke Servlet.service() for
servlet [jsp] in context with path [/test] threw exception [/el-misc.jsp
(line: 44, column: 21) attribute for " is not properly terminated] with
root cause
 org.apache.jasper.JasperException: /el-misc.jsp (line: 44, column: 21)
attribute for " is not properly terminated
at
org.apache.jasper.compiler.DefaultErrorHandler.jspError(DefaultErrorHandler.java:41)
at
org.apache.jasper.compiler.ErrorDispatcher.dispatch(ErrorDispatcher.java:275)
at
org.apache.jasper.compiler.ErrorDispatcher.jspError(ErrorDispatcher.java:91)
at org.apache.jasper.compiler.Parser.parseAttributeValue(Parser.java:270)
at org.apache.jasper.compiler.Parser.parseAttribute(Parser.java:235)
at org.apache.jasper.compiler.Parser.parseAttributes(Parser.java:163)
at org.apache.jasper.compiler.Parser.parseAttributes(Parser.java:154)
at org.apache.jasper.compiler.Parser.parseCustomTag(Parser.java:1236)
at org.apache.jasper.compiler.Parser.parseElements(Parser.java:1455)
at org.apache.jasper.compiler.Parser.parse(Parser.java:139)
at
org.apache.jasper.compiler.ParserController.doParse(ParserController.java:227)


> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> For additional commands, e-mail: dev-h...@tomcat.apache.org
>

Reply via email to