https://bz.apache.org/bugzilla/show_bug.cgi?id=57434
Jeremy Boynes changed:
What|Removed |Added
Status|NEEDINFO|RESOLVED
Resolution|---
https://bz.apache.org/bugzilla/show_bug.cgi?id=57434
--- Comment #11 from Tomas Hofman ---
I can no longer reproduce the issue with patched version of jstlel. Thanks
Jeremy!
--
You are receiving this mail because:
You are the assignee for the bug.
---
https://bz.apache.org/bugzilla/show_bug.cgi?id=57434
Tomas Hofman changed:
What|Removed |Added
CC||thof...@redhat.com
--
You are receivin
https://bz.apache.org/bugzilla/show_bug.cgi?id=57434
--- Comment #10 from Tomas Hofman ---
Yes, I think I do, but I'm afraid it's gonna involve Wildfly. I will check it
out and get back here.
--
You are receiving this mail because:
You are the assignee for the bug.
-
https://bz.apache.org/bugzilla/show_bug.cgi?id=57434
Jeremy Boynes changed:
What|Removed |Added
Status|NEW |NEEDINFO
--- Comment #9 from Jeremy Bo
https://bz.apache.org/bugzilla/show_bug.cgi?id=57434
--- Comment #8 from Jeremy Boynes ---
Created attachment 35845
--> https://bz.apache.org/bugzilla/attachment.cgi?id=35845&action=edit
Adds new public method that allows the cache to be bypassed when parsing
expressions.
Based on Tomas's PR,
https://bz.apache.org/bugzilla/show_bug.cgi?id=57434
--- Comment #7 from Tomas Hofman ---
Jeremy, what do you think about the PR?
https://github.com/jboss/jboss-jstl-api_spec/pull/16/files
If you don't want to change current behaviour, what about at least preventing
the NPE from happening (th
https://bz.apache.org/bugzilla/show_bug.cgi?id=57434
--- Comment #6 from Tomas Hofman ---
Linking the JBoss issue for reference: https://issues.jboss.org/browse/JBEE-183
--
You are receiving this mail because:
You are the assignee for the bug.
---
https://bz.apache.org/bugzilla/show_bug.cgi?id=57434
--- Comment #5 from Tomas Hofman ---
Sorry for not elaborating before - in fact my main motivation was that
currently the parseExpressionString() method can throw a NPE because of the
race, which breaks the request processing. Moreover when the
https://bz.apache.org/bugzilla/show_bug.cgi?id=57434
--- Comment #4 from Jeremy Boynes ---
The current implementation disables caching while expressions are being
validated during the translation process. This has the side effect of disabling
the cache for evaluations that occur concurrently i.e.
https://bz.apache.org/bugzilla/show_bug.cgi?id=57434
--- Comment #3 from Jeremy Boynes ---
I believe this patch reduces the window for the race but does not fully close
it as another thread could still modify the field between when it is set and
then read. It would be unlikely due to variable cac
https://bz.apache.org/bugzilla/show_bug.cgi?id=57434
--- Comment #2 from Tomas Hofman ---
Created attachment 35602
--> https://bz.apache.org/bugzilla/attachment.cgi?id=35602&action=edit
Patch
--
You are receiving this mail because:
You are the assignee for the bug.
---
https://bz.apache.org/bugzilla/show_bug.cgi?id=57434
--- Comment #1 from Tomas Hofman ---
This concurrency issue was also hit in EAP/Wildfly. It is an issue even if the
library is included inside a web application, e.g. when two different JSPs are
requested and evaluated concurrently.
Tested wit
13 matches
Mail list logo
