Add a handful of TODO notes

Author: akx

upcloud_api/cloud_manager/base.py

@@ -29,12 +29,13 @@ def request(self, method, endpoint, body=None, params=None, timeout=-1, request_
         if method not in {'GET', 'POST', 'PUT', 'PATCH', 'DELETE'}:
             raise Exception('Invalid/Forbidden HTTP method')
 
+        # TODO: revise the semantics of `request_to_api` and where it is set to False
         url = 'https://api.upcloud.com/' + self.api_v + endpoint if request_to_api else endpoint
-        headers = {
-            'Authorization': self.token,
-            'User-Agent': self._get_user_agent(),
-            'Content-Type': ('application/json' if request_to_api else 'application/octet-stream'),
-        }
+        headers = {'Authorization': self.token, 'User-Agent': self._get_user_agent()}
+
+        headers['Content-Type'] = (
+            'application/json' if request_to_api else 'application/octet-stream'
+        )
 
         if body and request_to_api:
             data = json.dumps(body)

upcloud_api/cloud_manager/firewall_mixin.py

@@ -17,6 +17,7 @@ class FirewallManager:
     directly.
     """
 
+    # TODO: server_instance is unused?
     def get_firewall_rule(self, server_uuid, firewall_rule_position, server_instance=None):
         """
         Return a FirewallRule object based on server uuid and rule position.

upcloud_api/cloud_manager/network_mixin.py

@@ -57,7 +57,7 @@ def create_network(
                 },
             }
         }
-
+        # TODO: fix duplication c.f. modify_network
         if router:
             body['network']['router'] = router
         if dhcp_default_route:

upcloud_api/server.py

@@ -410,6 +410,7 @@ def to_dict(self):
         del fields['cloud_manager']
         return fields
 
+    # TODO: strict is unused?
     def get_ip(self, access='public', addr_family=None, strict=None):
         """
         Return the server's IP address.