Issue Details (XML | Word | Printable)

Key: XW-395
Type: Improvement Improvement
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Don Brown
Reporter: Don Brown
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
XWork

RuntimeExceptions thrown by OGNL should not be immediately caught

Created: 25/Jun/06 12:29 AM   Updated: 14/Nov/06 11:34 PM
Component/s: None
Affects Version/s: 1.1.3
Fix Version/s: 2.0


 Description  « Hide
[Original bug - http://issues.apache.org/struts/browse/WW-662]

does anyone think that tags/ognl should re-throw runtime exceptions and let the container catch them instead of just just logging them how it is now and continuing to load the page normally?

I was using a previous version of webwork2 and this wasnt happening before.

what im doing is loading a proxied object from hibernate in my action then going to the view to display it but its throwing an ObjectNotFoundException when the tags try to access the proxy, I would have thought that hibernate would throw that ONFE when I did the actual session.load() instead of when I actually tried to use the proxy but thats not the case.



[ERROR] CompoundRootAccessor - Caught an Ognl exception while getting property ex
ognl.OgnlException: ex [java.lang.RuntimeException: getEx()]
        at ognl.OgnlRuntime.getMethodValue(OgnlRuntime.java:908)
        at ognl.ObjectPropertyAccessor.getPossibleProperty(ObjectPropertyAccessor.java:54)
        at ognl.ObjectPropertyAccessor.getProperty(ObjectPropertyAccessor.java:122)
        at ognl.OgnlRuntime.getProperty(OgnlRuntime.java:1616)
        at com.opensymphony.xwork.util.CompoundRootAccessor.getProperty(CompoundRootAccessor.java:95)
        at ognl.OgnlRuntime.getProperty(OgnlRuntime.java:1616)
        at ognl.ASTProperty.getValueBody(ASTProperty.java:96)
        at ognl.SimpleNode.evaluateGetValueBody(SimpleNode.java:171)
        at ognl.SimpleNode.getValue(SimpleNode.java:193)
        at ognl.Ognl.getValue(Ognl.java:333)
        at com.opensymphony.xwork.util.OgnlValueStack.findValue(OgnlValueStack.java:168)
        at com.opensymphony.webwork.views.jsp.WebWorkTagSupport.findValue(WebWorkTagSupport.java:51)
        at com.opensymphony.webwork.views.jsp.PropertyTag.doStartTag(PropertyTag.java:58)
        at __jspPage0_WEB_INF_views_index_jsp._jspService(__jspPage0_WEB_INF_views_index_jsp.java:29)
        at com.orionserver.http.OrionHttpJspPage.service(.:70)
        at com.evermind._ay._rkb(.:5741)
        at com.evermind.server.http.JSPServlet.service(.:31)
        at com.evermind._hb.doFilter(.:59)
        at org.springframework.web.filter.OncePerRequestFilter.doFilter(OncePerRequestFilter.java:68)
        at com.evermind._ha.doFilter(.:16)
        at com.opensymphony.webwork.lifecycle.RequestLifecycleFilter.doFilter(RequestLifecycleFilter.java:75)
        at com.evermind._cub._pod(.:383)
        at com.evermind._cub._bmc(.:177)
        at com.evermind._cub.forward(.:147)
        at com.opensymphony.webwork.dispatcher.ServletDispatcherResult.doExecute(ServletDispatcherResult.java:80)
        at com.opensymphony.webwork.dispatcher.WebWorkResultSupport.execute(WebWorkResultSupport.java:115)
        at com.opensymphony.xwork.DefaultActionInvocation.executeResult(DefaultActionInvocation.java:261)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:185)
        at com.opensymphony.xwork.interceptor.DefaultWorkflowInterceptor.intercept(DefaultWorkflowInterceptor.java:55)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.interceptor.AroundInterceptor.intercept(AroundInterceptor.java:34)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.interceptor.AroundInterceptor.intercept(AroundInterceptor.java:34)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.interceptor.AroundInterceptor.intercept(AroundInterceptor.java:34)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.interceptor.AroundInterceptor.intercept(AroundInterceptor.java:34)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.interceptor.AroundInterceptor.intercept(AroundInterceptor.java:34)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.interceptor.AroundInterceptor.intercept(AroundInterceptor.java:34)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at o2.interceptor.SetServiceLocator.intercept(SetServiceLocator.java:42)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.interceptor.AroundInterceptor.intercept(AroundInterceptor.java:34)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.interceptor.TimerInterceptor.intercept(TimerInterceptor.java:63)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.DefaultActionProxy.execute(DefaultActionProxy.java:115)
        at com.opensymphony.webwork.dispatcher.ServletDispatcher.serviceAction(ServletDispatcher.java:278)
        at com.opensymphony.webwork.dispatcher.ServletDispatcher.service(ServletDispatcher.java:248)
        at javax.servlet.http.HttpServlet.service(HttpServlet.java:336)
        at com.evermind._hb.doFilter(.:59)
        at o2.servlet.TimerFilter.doFilter(TimerFilter.java:14)
        at com.evermind._ha.doFilter(.:16)
        at org.springframework.orm.hibernate.support.OpenSessionInViewFilter.doFilterInternal(OpenSessionInViewFilter.java:117)
        at org.springframework.web.filter.OncePerRequestFilter.doFilter(OncePerRequestFilter.java:73)
        at com.evermind._ha.doFilter(.:20)
        at com.opensymphony.webwork.lifecycle.RequestLifecycleFilter.doFilter(RequestLifecycleFilter.java:75)
        at com.evermind._ha.doFilter(.:20)
        at com.opensymphony.module.sitemesh.filter.PageFilter.parsePage(Unknown Source)
        at com.opensymphony.module.sitemesh.filter.PageFilter.doFilter(Unknown Source)
        at com.evermind._cub._pod(.:383)
        at com.evermind._cub._bmc(.:177)
        at com.evermind._ax._ltc(.:666)
        at com.evermind._ax._uab(.:191)
        at com.evermind._bf.run(.:62)
/-- Encapsulated exception ------------\
java.lang.RuntimeException: getEx()
        at o2.action.AbstractAction.getEx(AbstractAction.java:53)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
        at java.lang.reflect.Method.invoke(Unknown Source)
        at ognl.OgnlRuntime.invokeMethod(OgnlRuntime.java:491)
        at ognl.OgnlRuntime.getMethodValue(OgnlRuntime.java:904)
        at ognl.ObjectPropertyAccessor.getPossibleProperty(ObjectPropertyAccessor.java:54)
        at ognl.ObjectPropertyAccessor.getProperty(ObjectPropertyAccessor.java:122)
        at ognl.OgnlRuntime.getProperty(OgnlRuntime.java:1616)
        at com.opensymphony.xwork.util.CompoundRootAccessor.getProperty(CompoundRootAccessor.java:95)
        at ognl.OgnlRuntime.getProperty(OgnlRuntime.java:1616)
        at ognl.ASTProperty.getValueBody(ASTProperty.java:96)
        at ognl.SimpleNode.evaluateGetValueBody(SimpleNode.java:171)
        at ognl.SimpleNode.getValue(SimpleNode.java:193)
        at ognl.Ognl.getValue(Ognl.java:333)
        at com.opensymphony.xwork.util.OgnlValueStack.findValue(OgnlValueStack.java:168)
        at com.opensymphony.webwork.views.jsp.WebWorkTagSupport.findValue(WebWorkTagSupport.java:51)
        at com.opensymphony.webwork.views.jsp.PropertyTag.doStartTag(PropertyTag.java:58)
        at __jspPage0_WEB_INF_views_index_jsp._jspService(__jspPage0_WEB_INF_views_index_jsp.java:29)
        at com.orionserver.http.OrionHttpJspPage.service(.:70)
        at com.evermind._ay._rkb(.:5741)
        at com.evermind.server.http.JSPServlet.service(.:31)
        at com.evermind._hb.doFilter(.:59)
        at org.springframework.web.filter.OncePerRequestFilter.doFilter(OncePerRequestFilter.java:68)
        at com.evermind._ha.doFilter(.:16)
        at com.opensymphony.webwork.lifecycle.RequestLifecycleFilter.doFilter(RequestLifecycleFilter.java:75)
        at com.evermind._cub._pod(.:383)
        at com.evermind._cub._bmc(.:177)
        at com.evermind._cub.forward(.:147)
        at com.opensymphony.webwork.dispatcher.ServletDispatcherResult.doExecute(ServletDispatcherResult.java:80)
        at com.opensymphony.webwork.dispatcher.WebWorkResultSupport.execute(WebWorkResultSupport.java:115)
        at com.opensymphony.xwork.DefaultActionInvocation.executeResult(DefaultActionInvocation.java:261)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:185)
        at com.opensymphony.xwork.interceptor.DefaultWorkflowInterceptor.intercept(DefaultWorkflowInterceptor.java:55)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.interceptor.AroundInterceptor.intercept(AroundInterceptor.java:34)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.interceptor.AroundInterceptor.intercept(AroundInterceptor.java:34)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.interceptor.AroundInterceptor.intercept(AroundInterceptor.java:34)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.interceptor.AroundInterceptor.intercept(AroundInterceptor.java:34)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.interceptor.AroundInterceptor.intercept(AroundInterceptor.java:34)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.interceptor.AroundInterceptor.intercept(AroundInterceptor.java:34)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at o2.interceptor.SetServiceLocator.intercept(SetServiceLocator.java:42)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.interceptor.AroundInterceptor.intercept(AroundInterceptor.java:34)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.interceptor.TimerInterceptor.intercept(TimerInterceptor.java:63)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.DefaultActionProxy.execute(DefaultActionProxy.java:115)
        at com.opensymphony.webwork.dispatcher.ServletDispatcher.serviceAction(ServletDispatcher.java:278)
        at com.opensymphony.webwork.dispatcher.ServletDispatcher.service(ServletDispatcher.java:248)
        at javax.servlet.http.HttpServlet.service(HttpServlet.java:336)
        at com.evermind._hb.doFilter(.:59)
        at o2.servlet.TimerFilter.doFilter(TimerFilter.java:14)
        at com.evermind._ha.doFilter(.:16)
        at org.springframework.orm.hibernate.support.OpenSessionInViewFilter.doFilterInternal(OpenSessionInViewFilter.java:117)
        at org.springframework.web.filter.OncePerRequestFilter.doFilter(OncePerRequestFilter.java:73)
        at com.evermind._ha.doFilter(.:20)
        at com.opensymphony.webwork.lifecycle.RequestLifecycleFilter.doFilter(RequestLifecycleFilter.java:75)
        at com.evermind._ha.doFilter(.:20)
        at com.opensymphony.module.sitemesh.filter.PageFilter.parsePage(Unknown Source)
        at com.opensymphony.module.sitemesh.filter.PageFilter.doFilter(Unknown Source)
        at com.evermind._cub._pod(.:383)
        at com.evermind._cub._bmc(.:177)
        at com.evermind._ax._ltc(.:666)
        at com.evermind._ax._uab(.:191)
        at com.evermind._bf.run(.:62)
\--------------------------------------/
[WARN] OgnlValueStack - Caught an exception while evaluating expression 'ex' against value stack
java.lang.RuntimeException: Caught an Ognl exception while getting property ex
        at com.opensymphony.xwork.util.CompoundRootAccessor.getProperty(CompoundRootAccessor.java:110)
        at ognl.OgnlRuntime.getProperty(OgnlRuntime.java:1616)
        at ognl.ASTProperty.getValueBody(ASTProperty.java:96)
        at ognl.SimpleNode.evaluateGetValueBody(SimpleNode.java:171)
        at ognl.SimpleNode.getValue(SimpleNode.java:193)
        at ognl.Ognl.getValue(Ognl.java:333)
        at com.opensymphony.xwork.util.OgnlValueStack.findValue(OgnlValueStack.java:168)
        at com.opensymphony.webwork.views.jsp.WebWorkTagSupport.findValue(WebWorkTagSupport.java:51)
        at com.opensymphony.webwork.views.jsp.PropertyTag.doStartTag(PropertyTag.java:58)
        at __jspPage0_WEB_INF_views_index_jsp._jspService(__jspPage0_WEB_INF_views_index_jsp.java:29)
        at com.orionserver.http.OrionHttpJspPage.service(.:70)
        at com.evermind._ay._rkb(.:5741)
        at com.evermind.server.http.JSPServlet.service(.:31)
        at com.evermind._hb.doFilter(.:59)
        at org.springframework.web.filter.OncePerRequestFilter.doFilter(OncePerRequestFilter.java:68)
        at com.evermind._ha.doFilter(.:16)
        at com.opensymphony.webwork.lifecycle.RequestLifecycleFilter.doFilter(RequestLifecycleFilter.java:75)
        at com.evermind._cub._pod(.:383)
        at com.evermind._cub._bmc(.:177)
        at com.evermind._cub.forward(.:147)
        at com.opensymphony.webwork.dispatcher.ServletDispatcherResult.doExecute(ServletDispatcherResult.java:80)
        at com.opensymphony.webwork.dispatcher.WebWorkResultSupport.execute(WebWorkResultSupport.java:115)
        at com.opensymphony.xwork.DefaultActionInvocation.executeResult(DefaultActionInvocation.java:261)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:185)
        at com.opensymphony.xwork.interceptor.DefaultWorkflowInterceptor.intercept(DefaultWorkflowInterceptor.java:55)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.interceptor.AroundInterceptor.intercept(AroundInterceptor.java:34)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.interceptor.AroundInterceptor.intercept(AroundInterceptor.java:34)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.interceptor.AroundInterceptor.intercept(AroundInterceptor.java:34)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.interceptor.AroundInterceptor.intercept(AroundInterceptor.java:34)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.interceptor.AroundInterceptor.intercept(AroundInterceptor.java:34)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.interceptor.AroundInterceptor.intercept(AroundInterceptor.java:34)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at o2.interceptor.SetServiceLocator.intercept(SetServiceLocator.java:42)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.interceptor.AroundInterceptor.intercept(AroundInterceptor.java:34)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.interceptor.TimerInterceptor.intercept(TimerInterceptor.java:63)
        at com.opensymphony.xwork.DefaultActionInvocation.invoke(DefaultActionInvocation.java:163)
        at com.opensymphony.xwork.DefaultActionProxy.execute(DefaultActionProxy.java:115)
        at com.opensymphony.webwork.dispatcher.ServletDispatcher.serviceAction(ServletDispatcher.java:278)
        at com.opensymphony.webwork.dispatcher.ServletDispatcher.service(ServletDispatcher.java:248)
        at javax.servlet.http.HttpServlet.service(HttpServlet.java:336)
        at com.evermind._hb.doFilter(.:59)
        at o2.servlet.TimerFilter.doFilter(TimerFilter.java:14)
        at com.evermind._ha.doFilter(.:16)
        at org.springframework.orm.hibernate.support.OpenSessionInViewFilter.doFilterInternal(OpenSessionInViewFilter.java:117)
        at org.springframework.web.filter.OncePerRequestFilter.doFilter(OncePerRequestFilter.java:73)
        at com.evermind._ha.doFilter(.:20)
        at com.opensymphony.webwork.lifecycle.RequestLifecycleFilter.doFilter(RequestLifecycleFilter.java:75)
        at com.evermind._ha.doFilter(.:20)
        at com.opensymphony.module.sitemesh.filter.PageFilter.parsePage(Unknown Source)
        at com.opensymphony.module.sitemesh.filter.PageFilter.doFilter(Unknown Source)
        at com.evermind._cub._pod(.:383)
        at com.evermind._cub._bmc(.:177)
        at com.evermind._ax._ltc(.:666)
        at com.evermind._ax._uab(.:191)
        at com.evermind._bf.run(.:62)
[INFO] LoggingInterceptor - Finishing execution stack for action index

 All   Comments   Change History      Sort Order: Ascending order - Click to sort in descending order
[ Permlink | « Hide ]
Alexandru Popescu added a comment - 25/Jun/06 02:49 AM
IMO this should happen only in DEV mode, but not otherwise.

./alex
--
.w( the_mindstorm )p.
---
(http://themindstorms.blogspot.com)

[ Permlink | « Hide ]
Don Brown added a comment - 14/Nov/06 11:34 PM
Should be fixed with better ognl exception handling. Please reopen if not the case.

Don Brown made changes - 14/Nov/06 11:34 PM
Field Original Value New Value
Assignee Don Brown [ mrdon ]
Resolution Fixed [ 1 ]
Status Open [ 1 ] Resolved [ 5 ]

Powered by a free Atlassian JIRA open source license for OpenSymphony. Try JIRA - bug tracking software for your team.
Atlassian JIRA the Professional Issue Tracker. (Enterprise Edition, Version: 3.13.2-#335) - Bug/feature request - Atlassian news - Contact Administrators