remove unnecessary check from firewall, add comment attribute

Author: Elias Nygren

upcloud_api/firewall.py

@@ -18,6 +18,7 @@ class FirewallRule(object):
         'action': 'drop',
         'direction': 'in',
         'family': 'IPv4',
+        'comment': '',
         'destination_address_end': None,
         'destination_address_start': None,
         'destination_port_end': None,
@@ -40,9 +41,6 @@ def __init__(self, **kwargs):
 
         # set object attributes from params
         for key in kwargs:
-            if key not in self.attributes:
-                self._invalid_key_err(key)
-
             setattr(self, key, kwargs[key])
 
         # set defaults (if need be) where the default is not None
@@ -82,16 +80,6 @@ def destroy(self):
         )
 
 
-    def _invalid_key_err(self, key):
-        """
-        Raise exception on invalid parameters given to __init__.
-        """
-        attr_list = list(self.attributes.keys())
-        raise Exception(
-            "invalid parameter to FirewallRule, '{key}' is not in {attributes}"
-            .format(key=key, attributes=attr_list)
-        )
-
     def _associate_with_server(self, server):
         """
         Internal function used by Server to associate itself with the FirewallRule.