aboutsummaryrefslogtreecommitdiff
path: root/web/apache
diff options
context:
space:
mode:
authorBoris Kolpackov <boris@codesynthesis.com>2015-12-17 19:20:41 +0200
committerBoris Kolpackov <boris@codesynthesis.com>2015-12-18 05:50:31 +0200
commit64e889f0ca84121a4e92840f9283cf9cf36279e6 (patch)
tree747ace5fbecd7cf854ce9bdbf02b4cabb25454e1 /web/apache
parent13a2e1a89d575332ae4155b22021306891c010de (diff)
Bugfix: brep::module::handle asserts the module is loaded
Diffstat (limited to 'web/apache')
-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);
}