litvinovg/vitro
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

checking for null RequestDispatcher in PageRoutingFitler

Browse source
This commit is contained in:
briancaruso 2011-07-25 14:54:03 +00:00
parent b03c4399c6
commit 47d8742503
1 changed files with 7 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

8
webapp/src/edu/cornell/mannlib/vitro/webapp/filters/PageRoutingFilter.java
View file

@ -81,7 +81,13 @@ public class PageRoutingFilter implements Filter{
String controllerName = getControllerToForwardTo(req, pageUri, pageDao); String controllerName = getControllerToForwardTo(req, pageUri, pageDao);
log.debug(path + " is being forwarded to controller " + controllerName); log.debug(path + " is being forwarded to controller " + controllerName);
RequestDispatcher rd = filterConfig.getServletContext().getNamedDispatcher( controllerName ); RequestDispatcher rd = filterConfig.getServletContext().getNamedDispatcher( controllerName );
if( rd == null ){
log.error(path + " should be forwarded to controller " + controllerName + " but there " +
"is no servlet named that defined for the web application in web.xml");
//TODO: what should be done in this case?
}
rd.forward(req, response); rd.forward(req, response);
}else if( "/".equals( path ) || path.isEmpty() ){ }else if( "/".equals( path ) || path.isEmpty() ){
log.debug("url '" +path + "' is being forward to home controller" ); log.debug("url '" +path + "' is being forward to home controller" );