Bugfix: brep::module::handle asserts the module is loaded

1 files changed, 8 insertions, 1 deletions

diff --git a/web/apache/service.cxx b/web/apache/service.cxx
index 1741af3..c6f830b 100644
--- a/web/apache/service.cxx
+++ b/web/apache/service.cxx
@@ -123,7 +123,14 @@ namespace web
       {
         l.write (nullptr, 0, func_name.c_str (), APLOG_EMERG, e.what ());
 
-        // Terminate the root apache process.
+        // Terminate the root apache process. Indeed we can only try to
+        // terminate the process, and most likely will fail in a production
+        // environment where the apache root process usually runs under root
+        // and worker processes run under some other user. This is why the
+        // implementation should consider the possibility of not being
+        // initialized at the time of HTTP request processing. In such a case
+        // it should respond with an internal server error (500 HTTP status),
+        // reporting misconfiguration.
         //
         ::kill (::getppid (), SIGTERM);
       }