Ok, fixed for real this time. The form parser was defaulting to UTF-8 when no charset was
present in the request.
Stuart
----- Original Message -----
From: "Stuart Douglas" <sdouglas(a)redhat.com>
To: "Wolfgang Knauf" <wolfgang.knauf(a)gmx.de>
Cc: undertow-dev(a)lists.jboss.org
Sent: Thursday, 29 August, 2013 2:39:35 PM
Subject: Re: [undertow-dev] Deadlock in JSP page on Wilflfy 8.0 Alpha 4 with umlauts
----- Original Message -----
> From: "Wolfgang Knauf" <wolfgang.knauf(a)gmx.de>
> To: "Stuart Douglas" <sdouglas(a)redhat.com>
> Cc: undertow-dev(a)lists.jboss.org
> Sent: Thursday, 29 August, 2013 2:30:20 PM
> Subject: Aw: Re: [undertow-dev] Deadlock in JSP page on Wilflfy 8.0 Alpha 4
> with umlauts
>
> Hi Stuart,
> thanks for the fast fix! The question mark is probably some kind of
> standard
> for non displayable strings ;-). Maybe also a log message, to trace down
> the
> error better?
> Any chance for me to test this against WildFly 8 Alpha 4 (without building
> it
> myself)?
Unfortunatly not yet. Once I release Undertow (probably next week) it will be
possible to get a nightly build.
> I assume that on my JSP page which handles the request, I have to do
> something like this:
> request.setCharacterEncoding ("ISO-8859-1");
This is the default character set. Hmm, I need to investigate what is going
on here, this is just using the standard charset encoder for ISO-8859-1, so
I am not sure why it is reporting those characters as unmappable.
Stuart
> This should hopefully write the correct characters to the database. I will
> do
> more testing this evening.
> On previous JBoss versions, my sample worked without those modifications.
> Did
> the Undertow migration include some change to character encoding handling?
Hmm, I will need to investigate this, and see how this was handled in
previous versions.
> If yes this might be an issue with backwards compatibility.
> Best regards
> Wolfgang
> Gesendet: Donnerstag, 29. August 2013 um 11:16 Uhr
> Von: "Stuart Douglas" <sdouglas(a)redhat.com>
> An: "Wolfgang Knauf" <wolfgang.knauf(a)gmx.de>
> Cc: undertow-dev(a)lists.jboss.org
> Betreff: Re: [undertow-dev] Deadlock in JSP page on Wilflfy 8.0 Alpha 4
> with
> umlauts
> I have thought about this a bit, and changed it so now it just writes a
> question mark on unmappable characters.
>
> It seems that this would be the most user friendly behaviour.
>
> Stuart
>
> ----- Original Message -----
> > From: "Stuart Douglas" <sdouglas(a)redhat.com>
> > To: "Wolfgang Knauf" <wolfgang.knauf(a)gmx.de>
> > Cc: undertow-dev(a)lists.jboss.org
> > Sent: Thursday, 29 August, 2013 11:00:14 AM
> > Subject: Re: [undertow-dev] Deadlock in JSP page on Wilflfy 8.0 Alpha 4
> > with umlauts
> >
> > This should be fixed in Undertow upstream. The writer will now error out
> > properly on unmappable characters. You will need to set a response
> > charset
> > of UTF8 (or some other appropriate charset) in order for this to work
> > correctly.
> >
> > The only thing I am not sure about is if this should be a fatal error, or
> > if
> > we should just write the unmappable characters directly to the output.
> >
> > Stuart
> >
> > ----- Original Message -----
> > > From: "Stuart Douglas" <sdouglas(a)redhat.com>
> > > To: "Wolfgang Knauf" <wolfgang.knauf(a)gmx.de>
> > > Cc: undertow-dev(a)lists.jboss.org
> > > Sent: Wednesday, 28 August, 2013 10:33:35 PM
> > > Subject: Re: [undertow-dev] Deadlock in JSP page on Wilflfy 8.0 Alpha 4
> > > with umlauts
> > >
> > > Thanks for reporting this, I will look into it.
> > >
> > > Stuart
> > >
> > > ----- Original Message -----
> > > > From: "Wolfgang Knauf" <wolfgang.knauf(a)gmx.de>
> > > > To: undertow-dev(a)lists.jboss.org
> > > > Sent: Wednesday, 28 August, 2013 10:28:24 PM
> > > > Subject: [undertow-dev] Deadlock in JSP page on Wilflfy 8.0 Alpha 4
> > > > with
> > > > umlauts
> > > >
> > > > Hi all,
> > > >
> > > > hope this is the right place and the correct component to post a
> > > > possible Undertow bug:
> > > > I have a simple EAR application, used to test the JavaEE feature of
a
> > > > one-to-many-relationship. It contains an EJB jar and a WAR file,
> > > > bundled
> > > > in an EAR file.
> > > >
> > > > Whenever I add an umlaut in an input field and submit the form,
> > > > WildFly
> > > > hangs up while processing the request to display the result page
> > > > (which
> > > > should display a list of my entities). As this happens when
> > > > generating
> > > > the JSP page, I assume an Undertow issue.
> > > >
> > > > To reproduce:
> > > > 1) deploy "KuchenZutat.ear" to WildFly (it uses the
> > > > "java:jboss/datasources/ExampleDS", so no configuration
changes
> > > > needed).
> > > > 2) browse to "http://localhost:8080/KuchenZutatWeb/"
> > > > 3) Klick "Neu"
> > > > 4) enter something with an umlaut, e.g. "Käsekuchen"
> > > > 5) click OK
> > > > Now the server hangs.
> > > >
> > > > It works when avoiding umlauts.
> > > >
> > > > Here are two stack traces, which I took by debugging the server with
> > > > Eclipse and breaking into debugger at random times. Note that the
> > > > thread
> > > > name is different in the two stack traces - I took them at different
> > > > server runs.
> > > > "CoderResult.unmappableForLength()" seems to be the place
for the
> > > > endless loop.
> > > >
> > > >
> > > > Thread [default task-3] (Suspended)
> > > > owns: CoderResult$2 (id=285)
> > > > HashMap<K,V>.hash(Object) line: not available
> > > > HashMap<K,V>.getEntry(Object) line: not available
> > > > HashMap<K,V>.get(Object) line: not available
> > > > CoderResult$2(CoderResult$Cache).get(int) line: not available
> > > > CoderResult$Cache.access$200(CoderResult$Cache, int) line: not
> > > > available
> > > > CoderResult.unmappableForLength(int) line: not available
> > > > Surrogate$Parser.unmappableResult() line: not available
> > > > ISO_8859_1$Encoder.encodeBufferLoop(CharBuffer, ByteBuffer) line:
not
> > > > available
> > > > ISO_8859_1$Encoder.encodeLoop(CharBuffer, ByteBuffer) line: not
> > > > available
> > > > ISO_8859_1$Encoder(CharsetEncoder).encode(CharBuffer, ByteBuffer,
> > > > boolean) line: not available
> > > > ServletPrintWriter.write(CharBuffer) line: 58
> > > > ServletPrintWriter.write(char[], int, int) line: 76
> > > > ServletPrintWriterDelegate.write(char[], int, int) line: 96
> > > > JspWriterImpl.flushBuffer() line: 120
> > > > PageContextImpl.release() line: 188
> > > > JspFactoryImpl.internalReleasePageContext(PageContext) line: 115
> > > > JspFactoryImpl.releasePageContext(PageContext) line: 74
> > > > index.jsp line: not available
> > > > index_jsp(HttpJspBase).service(HttpServletRequest,
> > > > HttpServletResponse)
> > > > line: 69
> > > > index_jsp(HttpServlet).service(ServletRequest, ServletResponse)
line:
> > > > 790
> > > > JspServletWrapper.service(HttpServletRequest, HttpServletResponse,
> > > > boolean) line: 365
> > > > JspServlet.serviceJspFile(HttpServletRequest, HttpServletResponse,
> > > > String, Throwable, boolean) line: 321
> > > > JspServlet.service(HttpServletRequest, HttpServletResponse) line:
254
> > > > JspServlet(HttpServlet).service(ServletRequest, ServletResponse)
> > > > line:
> > > > 790
> > > > ServletHandler.handleRequest(HttpServerExchange) line: 87
> > > > FilterHandler.handleRequest(HttpServerExchange) line: 82
> > > > ServletSecurityRoleHandler.handleRequest(HttpServerExchange) line:
54
> > > > ServletDispatchingHandler.handleRequest(HttpServerExchange) line: 36
> > > > SecurityContextAssociationHandler.handleRequest(HttpServerExchange)
> > > > line: 78
> > > > PredicateHandler.handleRequest(HttpServerExchange) line: 25
> > > > PredicateHandler.handleRequest(HttpServerExchange) line: 25
> > > > ServletInitialHandler.dispatchRequest(HttpServerExchange,
> > > > ServletRequestContext, ServletChain, DispatcherType) line: 196
> > > > ServletInitialHandler.dispatchToPath(HttpServerExchange,
> > > > ServletPathMatch, DispatcherType) line: 139
> > > > RequestDispatcherImpl.forward(ServletRequest, ServletResponse) line:
> > > > 156
> > > > KuchenEdit.jsp line: 40
> > > > KuchenEdit_jsp(HttpJspBase).service(HttpServletRequest,
> > > > HttpServletResponse) line: 69
> > > > KuchenEdit_jsp(HttpServlet).service(ServletRequest, ServletResponse)
> > > > line: 790
> > > > JspServletWrapper.service(HttpServletRequest, HttpServletResponse,
> > > > boolean) line: 365
> > > > JspServlet.serviceJspFile(HttpServletRequest, HttpServletResponse,
> > > > String, Throwable, boolean) line: 321
> > > > JspServlet.service(HttpServletRequest, HttpServletResponse) line:
254
> > > > JspServlet(HttpServlet).service(ServletRequest, ServletResponse)
> > > > line:
> > > > 790
> > > > ServletHandler.handleRequest(HttpServerExchange) line: 87
> > > > FilterHandler$FilterChainImpl.doFilter(ServletRequest,
> > > > ServletResponse)
> > > > line: 130
> > > > JsrWebSocketFilter.doFilter(ServletRequest, ServletResponse,
> > > > FilterChain) line: 136
> > > > ManagedFilter.doFilter(ServletRequest, ServletResponse, FilterChain)
> > > > line: 56
> > > > FilterHandler$FilterChainImpl.doFilter(ServletRequest,
> > > > ServletResponse)
> > > > line: 132
> > > > JsrWebSocketFilter.doFilter(ServletRequest, ServletResponse,
> > > > FilterChain) line: 136
> > > > ManagedFilter.doFilter(ServletRequest, ServletResponse, FilterChain)
> > > > line: 56
> > > > FilterHandler$FilterChainImpl.doFilter(ServletRequest,
> > > > ServletResponse)
> > > > line: 132
> > > > FilterHandler.handleRequest(HttpServerExchange) line: 85
> > > > ServletSecurityRoleHandler.handleRequest(HttpServerExchange) line:
56
> > > > ServletDispatchingHandler.handleRequest(HttpServerExchange) line: 36
> > > > SecurityContextAssociationHandler.handleRequest(HttpServerExchange)
> > > > line: 78
> > > > SSLInformationAssociationHandler.handleRequest(HttpServerExchange)
> > > > line: 113
> > > > AuthenticationCallHandler.handleRequest(HttpServerExchange) line: 52
> > > >
ServletConfidentialityConstraintHandler(AbstractConfidentialityHandler).handleRequest(HttpServerExchange)
> > > > line: 45
> > > > CachedAuthenticatedSessionHandler.handleRequest(HttpServerExchange)
> > > > line: 65
> > > > SecurityInitialHandler.handleRequest(HttpServerExchange) line: 70
> > > > PredicateHandler.handleRequest(HttpServerExchange) line: 25
> > > > SecurityContextCreationHandler.handleRequest(HttpServerExchange)
> > > > line:
> > > > 54
> > > > PredicateHandler.handleRequest(HttpServerExchange) line: 25
> > > > ServletInitialHandler.handleFirstRequest(HttpServerExchange,
> > > > ServletChain, ServletRequestContext, ServletRequest,
ServletResponse)
> > > > line: 207
> > > > ServletInitialHandler.dispatchRequest(HttpServerExchange,
> > > > ServletRequestContext, ServletChain, DispatcherType) line: 194
> > > > ServletInitialHandler.access$000(ServletInitialHandler,
> > > > HttpServerExchange, ServletRequestContext, ServletChain,
> > > > DispatcherType)
> > > > line: 72
> > > > ServletInitialHandler$1.handleRequest(HttpServerExchange) line: 128
> > > > HttpHandlers.executeRootHandler(HttpHandler, HttpServerExchange,
> > > > boolean) line: 36
> > > > HttpServerExchange$1.run() line: 628
> > > >
XnioWorker$TaskPool(ThreadPoolExecutor).runWorker(ThreadPoolExecutor$Worker)
> > > > line: not available
> > > > ThreadPoolExecutor$Worker.run() line: not available
> > > > Thread.run() line: not available
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > > Thread [default task-8] (Suspended)
> > > > CoderResult$Cache.access$200(CoderResult$Cache, int) line: not
> > > > available
> > > > CoderResult.unmappableForLength(int) line: not available
> > > > Surrogate$Parser.unmappableResult() line: not available
> > > > ISO_8859_1$Encoder.encodeBufferLoop(CharBuffer, ByteBuffer) line:
not
> > > > available
> > > > ISO_8859_1$Encoder.encodeLoop(CharBuffer, ByteBuffer) line: not
> > > > available
> > > > ISO_8859_1$Encoder(CharsetEncoder).encode(CharBuffer, ByteBuffer,
> > > > boolean) line: not available
> > > > ServletPrintWriter.write(CharBuffer) line: 58
> > > > ServletPrintWriter.write(char[], int, int) line: 76
> > > > ServletPrintWriterDelegate.write(char[], int, int) line: 96
> > > > JspWriterImpl.flushBuffer() line: 120
> > > > PageContextImpl.release() line: 188
> > > > JspFactoryImpl.internalReleasePageContext(PageContext) line: 115
> > > > JspFactoryImpl.releasePageContext(PageContext) line: 74
> > > > Kuchen.jsp line: not available
> > > > Kuchen_jsp(HttpJspBase).service(HttpServletRequest,
> > > > HttpServletResponse) line: 69
> > > > Kuchen_jsp(HttpServlet).service(ServletRequest, ServletResponse)
> > > > line:
> > > > 790
> > > > JspServletWrapper.service(HttpServletRequest, HttpServletResponse,
> > > > boolean) line: 365
> > > > JspServlet.serviceJspFile(HttpServletRequest, HttpServletResponse,
> > > > String, Throwable, boolean) line: 321
> > > > JspServlet.service(HttpServletRequest, HttpServletResponse) line:
254
> > > > JspServlet(HttpServlet).service(ServletRequest, ServletResponse)
> > > > line:
> > > > 790
> > > > ServletHandler.handleRequest(HttpServerExchange) line: 87
> > > > FilterHandler.handleRequest(HttpServerExchange) line: 82
> > > > ServletSecurityRoleHandler.handleRequest(HttpServerExchange) line:
54
> > > > ServletDispatchingHandler.handleRequest(HttpServerExchange) line: 36
> > > > SecurityContextAssociationHandler.handleRequest(HttpServerExchange)
> > > > line: 78
> > > > PredicateHandler.handleRequest(HttpServerExchange) line: 25
> > > > PredicateHandler.handleRequest(HttpServerExchange) line: 25
> > > > ServletInitialHandler.dispatchRequest(HttpServerExchange,
> > > > ServletRequestContext, ServletChain, DispatcherType) line: 196
> > > > ServletInitialHandler.dispatchToPath(HttpServerExchange,
> > > > ServletPathMatch, DispatcherType) line: 139
> > > > RequestDispatcherImpl.forward(ServletRequest, ServletResponse) line:
> > > > 156
> > > > KuchenEdit.jsp line: 43
> > > > KuchenEdit_jsp(HttpJspBase).service(HttpServletRequest,
> > > > HttpServletResponse) line: 69
> > > > KuchenEdit_jsp(HttpServlet).service(ServletRequest, ServletResponse)
> > > > line: 790
> > > > JspServletWrapper.service(HttpServletRequest, HttpServletResponse,
> > > > boolean) line: 365
> > > > JspServlet.serviceJspFile(HttpServletRequest, HttpServletResponse,
> > > > String, Throwable, boolean) line: 321
> > > > JspServlet.service(HttpServletRequest, HttpServletResponse) line:
254
> > > > JspServlet(HttpServlet).service(ServletRequest, ServletResponse)
> > > > line:
> > > > 790
> > > > ServletHandler.handleRequest(HttpServerExchange) line: 87
> > > > FilterHandler$FilterChainImpl.doFilter(ServletRequest,
> > > > ServletResponse)
> > > > line: 130
> > > > JsrWebSocketFilter.doFilter(ServletRequest, ServletResponse,
> > > > FilterChain) line: 136
> > > > ManagedFilter.doFilter(ServletRequest, ServletResponse, FilterChain)
> > > > line: 56
> > > > FilterHandler$FilterChainImpl.doFilter(ServletRequest,
> > > > ServletResponse)
> > > > line: 132
> > > > JsrWebSocketFilter.doFilter(ServletRequest, ServletResponse,
> > > > FilterChain) line: 136
> > > > ManagedFilter.doFilter(ServletRequest, ServletResponse, FilterChain)
> > > > line: 56
> > > > FilterHandler$FilterChainImpl.doFilter(ServletRequest,
> > > > ServletResponse)
> > > > line: 132
> > > > FilterHandler.handleRequest(HttpServerExchange) line: 85
> > > > ServletSecurityRoleHandler.handleRequest(HttpServerExchange) line:
56
> > > > ServletDispatchingHandler.handleRequest(HttpServerExchange) line: 36
> > > > SecurityContextAssociationHandler.handleRequest(HttpServerExchange)
> > > > line: 78
> > > > SSLInformationAssociationHandler.handleRequest(HttpServerExchange)
> > > > line: 113
> > > > AuthenticationCallHandler.handleRequest(HttpServerExchange) line: 52
> > > >
ServletConfidentialityConstraintHandler(AbstractConfidentialityHandler).handleRequest(HttpServerExchange)
> > > > line: 45
> > > > CachedAuthenticatedSessionHandler.handleRequest(HttpServerExchange)
> > > > line: 65
> > > > SecurityInitialHandler.handleRequest(HttpServerExchange) line: 70
> > > > PredicateHandler.handleRequest(HttpServerExchange) line: 25
> > > > SecurityContextCreationHandler.handleRequest(HttpServerExchange)
> > > > line:
> > > > 54
> > > > PredicateHandler.handleRequest(HttpServerExchange) line: 25
> > > > ServletInitialHandler.handleFirstRequest(HttpServerExchange,
> > > > ServletChain, ServletRequestContext, ServletRequest,
ServletResponse)
> > > > line: 207
> > > > ServletInitialHandler.dispatchRequest(HttpServerExchange,
> > > > ServletRequestContext, ServletChain, DispatcherType) line: 194
> > > > ServletInitialHandler.access$000(ServletInitialHandler,
> > > > HttpServerExchange, ServletRequestContext, ServletChain,
> > > > DispatcherType)
> > > > line: 72
> > > > ServletInitialHandler$1.handleRequest(HttpServerExchange) line: 128
> > > > HttpHandlers.executeRootHandler(HttpHandler, HttpServerExchange,
> > > > boolean) line: 36
> > > > HttpServerExchange$1.run() line: 628
> > > >
XnioWorker$TaskPool(ThreadPoolExecutor).runWorker(ThreadPoolExecutor$Worker)
> > > > line: not available
> > > > ThreadPoolExecutor$Worker.run() line: not available
> > > > Thread.run() line: not available
> > > >
> > > >
> > > > Best regards
> > > >
> > > > Wolfgang Knauf
> > > >
> > > > _______________________________________________
> > > > undertow-dev mailing list
> > > > undertow-dev(a)lists.jboss.org
> > > >
https://lists.jboss.org/mailman/listinfo/undertow-dev
> > >
> > > _______________________________________________
> > > undertow-dev mailing list
> > > undertow-dev(a)lists.jboss.org
> > >
https://lists.jboss.org/mailman/listinfo/undertow-dev
> >
> > _______________________________________________
> > undertow-dev mailing list
> > undertow-dev(a)lists.jboss.org
> >
https://lists.jboss.org/mailman/listinfo/undertow-dev
>
_______________________________________________
undertow-dev mailing list
undertow-dev(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/undertow-dev