On Tue, Jan 29, 2013 at 6:08 PM, Konstantin Kolinko <knst.koli...@gmail.com>wrote:
> 2013/1/30 Caldarale, Charles R <chuck.caldar...@unisys.com>: > >> From: Howard W. Smith, Jr. [mailto:smithh032...@gmail.com] > >> Subject: Latest-version TomEE 1.5.2-SNAPSHOT (Tomcat 7.0.34?) just died > on me, nothing in server logs > > > >> Sending this to tomee/openejb and tomcat user mailing lists. > > > > Cross-posting is really frowned upon. > > > >> Jan 29, 2013 10:27:25 AM org.apache.catalina.core.StandardWrapperValve > event > >> SEVERE: Servlet.service() for servlet [Push Servlet] in context with > path > >> [/mcmsweb] threw exception > >> java.lang.NumberFormatException: For input string: "(TomEE)/7" > >> at java.lang.NumberFormatException.forInputString(Unknown Source) > >> at java.lang.Integer.parseInt(Unknown Source) > >> at java.lang.Integer.valueOf(Unknown Source) > >> at > org.atmosphere.container.Tomcat7CometSupport.bz51881(Tomcat7CometSupport.java:141) > > > > Sure looks like an Atmosphere problem from here... > > > > +1. > > > https://github.com/Atmosphere/atmosphere/blob/master/modules/cpr/src/main/java/org/atmosphere/container/Tomcat7CometSupport.java#L129 > > That "bz51881" method... an example of what you should never do... > > It tries to parse a version string to "workaround" a bug that was > properly fixed more than a year ago, and fails horribly. > > It is a normal practice to modify the "server info" value, e.g. [1]. > It is useless to rely on it. It is no wonder that the one used by > TomEE is different from Tomcat. > > [1] http://tomcat.apache.org/tomcat-7.0-doc/security-howto.html#Valves > > Interesting, thanks for the response, Konstantin.
