https://issues.apache.org/bugzilla/show_bug.cgi?id=56461
            Bug ID: 56461
           Summary: New option to make context startup fail if a
                    load-on-startup servlet fails
           Product: Tomcat 7
           Version: 7.0.53
          Hardware: All
                OS: All
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: Catalina
          Assignee: dev@tomcat.apache.org
          Reporter: slaur...@apache.org

Created attachment 31563
  --> https://issues.apache.org/bugzilla/attachment.cgi?id=31563&action=edit
Patch for tomcat 8

Currently when a servlet with load-on-startup >=0 fails its startup, the
context startup is still considered as OK.
With some webapps (like spring-based ones with a DispatcherServlet), this makes
no sense at all : if the servlet failed its startup, the webapp is unuseable
and it would be more sensible to have tomcat mark the context as failed.

Such a behavior is not specified in JEE specs, so I propose to have a new
optional flag on Context and Host containers.

In the attached patch, the flag is named
failDeploymentIfServletLoadedOnStartupFails . If specified on Context, it
overrides the value potentially specified at the Host level.

The proposed patch for tomcat 8 passes checkstyle, provides a unit test and
updates documentation.
If it's OK with the patch, I'd also like to have it backported to tomcat 7.

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to