[GitHub] [tomcat] wilkinsona commented on pull request #456: Document conditions under which the AprLifecycleListener can be used …

[GitBox <gi...@apache.org>]

wilkinsona commented on pull request #456:
URL: https://github.com/apache/tomcat/pull/456#issuecomment-954908926


   I agree, @ChristopherSchultz. IMO, the second `AprLifecycleListener` should not be shutting down APR if it did nothing when asked to initialize it.
   
   The ordering in Spring Boot is the following:
   
   1. Start main Tomcat instance
   2. Start management Tomcat instance
   3. Stop management Tomcat instance
   4. Stop main Tomcat instance
   
   From an APR perspective, the following happens at the four steps above:
   
   1. APR is initialized in response to `BEFORE_INIT_EVENT`
   2. APR is already initialized so handling of `BEFORE_INIT_EVENT` is essentially a no-op
   3. APR is terminated in `AFTER_DESTROY_EVENT`
   
   The JVM crashes before we get to step 4 as APR is been ripped out from underneath the main Tomcat instance.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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