https://bz.apache.org/bugzilla/show_bug.cgi?id=62757
--- Comment #11 from Jordi Llach ---
Any ETA for Tomcat 8.5.35 ?
Thanks
J
--
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe,
https://bz.apache.org/bugzilla/show_bug.cgi?id=62757
Mark Thomas changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://bz.apache.org/bugzilla/show_bug.cgi?id=62757
Jordi Llach changed:
What|Removed |Added
OS|Mac OS X 10.1 |All
Hardware|PC
https://bz.apache.org/bugzilla/show_bug.cgi?id=62757
Jordi Llach changed:
What|Removed |Added
Version|8.5.34 |8.5.33
Severity|normal
https://bz.apache.org/bugzilla/show_bug.cgi?id=62757
--- Comment #9 from Jordi Llach ---
The way you've solved the issue in JspServletWrapper's loadTagFile method by
checking whether tagHandlerClass is null (in the second synchronized block) is
better than mine. Also, removing firstTime from
https://bz.apache.org/bugzilla/show_bug.cgi?id=62757
--- Comment #8 from Jordi Llach ---
Cool, I'll review it and test it tomorrow
--
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe,
https://bz.apache.org/bugzilla/show_bug.cgi?id=62757
--- Comment #7 from Mark Thomas ---
Created attachment 36186
--> https://bz.apache.org/bugzilla/attachment.cgi?id=36186=edit
Alternative patch
Thanks for the additional explanation. I agree with all of it. I had similar
concerns when I was
https://bz.apache.org/bugzilla/show_bug.cgi?id=62757
Mark Thomas changed:
What|Removed |Added
CC||jul...@beti.name
--- Comment #6 from
https://bz.apache.org/bugzilla/show_bug.cgi?id=62757
--- Comment #5 from Jordi Llach ---
Thanks for the feedback, I can perfectly understand the rationale of your
comment
As said the problem is in JspServletWrapper's loadTagFile method, it should
check whether it has to reload or not by only
https://bz.apache.org/bugzilla/show_bug.cgi?id=62757
--- Comment #4 from Mark Thomas ---
I can't tell which part of the patch is addressing this issue and which parts
are addressing other issues - or what those issues are. Given I've already
managed to introduce a regression here I'm hesitant to
https://bz.apache.org/bugzilla/show_bug.cgi?id=62757
--- Comment #3 from Jordi ---
Created attachment 36169
--> https://bz.apache.org/bugzilla/attachment.cgi?id=36169=edit
patch for the reported bug
I've found the problem, JspServletWrapper's loadTagFile should check whether it
has to reload
https://bz.apache.org/bugzilla/show_bug.cgi?id=62757
--- Comment #2 from Jordi ---
By deactivating "checkInterval" in JspServlet's init config param the problem
dissappears. Also notice that we set to false the "development" init param
I can consistently reproduce the problem with a huge
https://bz.apache.org/bugzilla/show_bug.cgi?id=62757
--- Comment #1 from Mark Thomas ---
Do you have a test case to reproduce?
--
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail:
13 matches
Mail list logo