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-unsubscr...@tomcat.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
