aboutsummaryrefslogtreecommitdiff
path: root/web/apache/service.cxx
diff options
context:
space:
mode:
Diffstat (limited to 'web/apache/service.cxx')
-rw-r--r--web/apache/service.cxx9
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);
}