Browse Source

[2211] added note about exception cases in remote config handler.

JINMEI Tatuya 12 years ago
parent
commit
a61ab28f80
1 changed files with 7 additions and 0 deletions
  1. 7 0
      src/bin/auth/main.cc

+ 7 - 0
src/bin/auth/main.cc

@@ -94,6 +94,13 @@ datasrcConfigHandler(AuthSrv* server, bool* first_time,
                      const isc::config::ConfigData&)
                      const isc::config::ConfigData&)
 {
 {
     assert(server != NULL);
     assert(server != NULL);
+
+    // Note: remote config handler is requested to be exception free.
+    // While the code below is not 100% exception free, such an exception
+    // is really fatal and the server should actually stop.  So we don't
+    // bother to catch them; the exception would be propagated to the
+    // top level of the server and terminate it.
+
     if (config->contains("classes")) {
     if (config->contains("classes")) {
         isc::datasrc::DataSrcClientListsPtr lists;
         isc::datasrc::DataSrcClientListsPtr lists;