Add exception for HTTP 429 status codes.

This commit adds support for handling HTTP 429 status codes and a new
exeption class for calling "To many requests" exceptions.

This class easily can be used to catch similar responses from other
DDNS providers too.

Signed-off-by: Dirk Wagner <dirk.wagner@ipfire.org>
Signed-off-by: Stefan Schantl <stefan.schantl@ipfire.org>

diff --git a/src/ddns/errors.py b/src/ddns/errors.py
index 58a5ba9beb0e0342d51d21dc7e140c3cbc591483..a8a201751f1111d41c2723681897e1261c467803 100644 (file)
--- a/src/ddns/errors.py
+++ b/src/ddns/errors.py
@@ -2,7 +2,7 @@
 ###############################################################################
 #                                                                             #
 # ddns - A dynamic DNS client for IPFire                                      #
-# Copyright (C) 2012 IPFire development team                                  #
+# Copyright (C) 2012-2017 IPFire development team                             #
 #                                                                             #
 # This program is free software: you can redistribute it and/or modify        #
 # it under the terms of the GNU General Public License as published by        #
@@ -161,6 +161,13 @@ class DDNSServiceUnavailableError(DDNSNetworkError):
        reason = N_("Service unavailable")
 
 
+class DDNSTooManyRequests(DDNSError):
+       """
+               Raised when too many requests occured.
+       """
+       reason = N_("Too many requests")
+
+
 class DDNSUpdateError(DDNSError):
        """
                Thrown when an update could not be

diff --git a/src/ddns/system.py b/src/ddns/system.py
index c268ba5cbae57dbbdd3aa1e1023ab96ca0cf41b1..67ea553b434bdcb7c96989328ea56422fe711973 100644 (file)
--- a/src/ddns/system.py
+++ b/src/ddns/system.py
@@ -194,6 +194,10 @@ class DDNSSystem(object):
                        elif e.code == 404:
                                raise DDNSNotFound(e.reason)
 
+                       # 429 - Too Many Requests
+                       elif e.code == 429:
+                               raise DDNSTooManyRequests(e.reason)
+
                        # 500 - Internal Server Error
                        elif e.code == 500:
                                raise DDNSInternalServerError(e.reason)