Fix 'occurred' typo

[bpdyndnsd] / src / service_zoneedit.cpp

diff --git a/src/service_zoneedit.cpp b/src/service_zoneedit.cpp
index 9365529..724e539 100644 (file)
--- a/src/service_zoneedit.cpp
+++ b/src/service_zoneedit.cpp
@@ -7,8 +7,8 @@
  * @license GPLv2
 */
 
-#include "service_zoneedit.h"
-#include "util.h"
+#include "service_zoneedit.hpp"
+#include "util.hpp"
 
 #include <time.h>
 #include <boost/foreach.hpp>
@@ -33,18 +33,23 @@ ServiceZoneedit::ServiceZoneedit()
  * @param _login The login name.
  * @param _password The corresponding password.
  */
-ServiceZoneedit::ServiceZoneedit(const string& _protocol, const string& _hostname, const string& _login, const string& _password, const Logger::Ptr& _logger, const int _update_interval, const int _max_updates_within_interval, const int _dns_cache_ttl, const string& _proxy, const int _proxy_port)
+ServiceZoneedit::ServiceZoneedit(const string& _protocol, const string& _hostname, const string& _login, const string& _password, const Logger::Ptr& _logger, const int _update_interval, const int _max_updates_within_interval, const int _max_equal_updates_in_succession, const int _dns_cache_ttl, const string& _proxy, const int _proxy_port)
 {
     if ( _update_interval == -1 )        // If _update_interval is default po::option_desc (not specified via config)
-        set_update_interval(0);              // use default protocol value
+        set_update_interval(15);              // use default protocol value
     else
         set_update_interval(_update_interval);
 
     if ( _max_updates_within_interval == -1 )
-        set_max_updates_within_interval(0);
+        set_max_updates_within_interval(3);
     else
         set_max_updates_within_interval(_max_updates_within_interval);
 
+    if ( _max_equal_updates_in_succession == -1 )
+        set_max_equal_updates_in_succession(2);
+    else
+        set_max_equal_updates_in_succession(_max_equal_updates_in_succession);
+
     if ( _dns_cache_ttl == -1 )
         set_dns_cache_ttl(60);
     else
@@ -57,8 +62,7 @@ ServiceZoneedit::ServiceZoneedit(const string& _protocol, const string& _hostnam
     set_logger(_logger);
 
     // create http helper class
-    HTTPHelper::Ptr _http_help(new HTTPHelper(_logger,_proxy,_proxy_port));
-    HTTPHelp.swap(_http_help);
+    HTTPHelp = HTTPHelper::Ptr(new HTTPHelper(_logger,_proxy,_proxy_port));
 
     BaseUrl = assemble_base_url(get_hostname());
 }
@@ -93,55 +97,52 @@ string ServiceZoneedit::assemble_base_url(const string& fqhn) const
 /**
  * Performs the Service update.
  * @param ip IP Address to set.
- * @return 0 if all is fine, -1 otherwise.
+ * @return UpdateOk if all is fine, GenericError otherwise.
  */
-int ServiceZoneedit::perform_update(const std::string& ip)
+Service::UpdateErrorCode ServiceZoneedit::perform_update(const std::string& ip)
 {
     string url = BaseUrl;
     url.append(ip);
 
-    long http_status_code = HTTPHelp->http_get(url);
-
-    get_logger()->print_http_status_code(url,http_status_code);
-
-    // HTTP operation completed successful.
-    // Now we have to parse the data received by curl,
-    // cause http status code is not significant for dyns update errors
-    if ( http_status_code == 200 )
+    if ( HTTPHelp->is_initialized() )
     {
-        // Get the received http data and parse the status code.
-        string curl_data = HTTPHelp->get_curl_data();
-        string status_code = Util::parse_status_code(curl_data);
+        long http_status_code = HTTPHelp->http_get(url);
 
-        if ( status_code == "200" )
+        get_logger()->print_http_status_code(url,http_status_code);
+
+        // HTTP operation completed successful.
+        // Now we have to parse the data received by curl,
+        // cause http status code is not significant for dyns update errors
+        if ( http_status_code == 200 )
+        {
+            // Get the received http data and parse the status code.
+            string curl_data = HTTPHelp->get_curl_data();
+            string status_code = Util::parse_status_code(curl_data);
+
+            if ( status_code == "200" )
+            {
+                return UpdateOk;
+            }
+            else
+            {
+                get_logger()->print_update_failure(url,curl_data);
+                return UpdateError;
+            }
+        }
+        else if ( http_status_code == 401 )
         {
-            return 0;
+            get_logger()->print_service_not_authorized(url,get_login(),get_password());
+            return NotAuth;
         }
         else
         {
-            get_logger()->print_update_failure(url,curl_data);
+            get_logger()->print_update_failure(url,http_status_code);
         }
     }
-    else if ( http_status_code == 401 )
-    {
-        get_logger()->print_service_not_authorized(url,get_login(),get_password());
-    }
     else
     {
-        get_logger()->print_update_failure(url,http_status_code);
+        get_logger()->print_httphelper_not_initialized();
+        HTTPHelp->re_initialize();
     }
-
-    return -1;
-}
-
-
-/**
- * Serialize function needed by boost/serialization to define which members should be stored as the object state.
- * @param ar Archive
- * @param version Version
- */
-template<class Archive>
-void ServiceZoneedit::serialize(Archive & ar, const unsigned int version)
-{
-    ar & boost::serialization::base_object<Service>(*this);
+    return GenericError;
 }