Version 3.4.2

CHANGELOG.md

@@ -1,6 +1,8 @@
 # Changelog
 
 ## master - CURRENT
+
+## 3.4.2 - 03/09/2020
 ### Modified
 * Relax versions constraints on Lexicon dependencies until there is a real need.
 

lexicon/providers/auto.py

@@ -223,7 +223,10 @@ def authenticate(self):
                 # ArgsConfigSource needs to be reprocessed to rescope the provided
                 # args to the delegate provider
                 new_dict = {}
-                for (key, value,) in config_source._parameters.items():
+                for (
+                    key,
+                    value,
+                ) in config_source._parameters.items():
                     if key.startswith(target_prefix):
                         new_param_name = re.sub("^{0}".format(target_prefix), "", key)
                         if provider_name not in new_dict:

lexicon/providers/easyname.py

@@ -20,13 +20,14 @@ def provider_parser(subparser):
         "--auth-username", help="Specify username used to authenticate"
     )
     subparser.add_argument(
-        "--auth-password", help="Specify password used to authenticate",
+        "--auth-password",
+        help="Specify password used to authenticate",
     )
 
 
 class Provider(BaseProvider):
     """
-        easyname provider
+    easyname provider
     """
 
     URLS = {

lexicon/providers/exoscale.py

@@ -53,7 +53,8 @@ def _create_record(self, rtype, name, content):
             record["prio"] = self._get_lexicon_option("priority")
 
         payload = self._post(
-            "/v1/domains/{0}/records".format(self.domain), {"record": record},
+            "/v1/domains/{0}/records".format(self.domain),
+            {"record": record},
         )
 
         status = "id" in payload.get("record", {})
@@ -75,7 +76,8 @@ def _list_records(self, rtype=None, name=None, content=None):
             name = self._relative_name(name)
             filter_query["name"] = name
         payload = self._get(
-            "/v1/domains/{0}/records".format(self.domain), query_params=filter_query,
+            "/v1/domains/{0}/records".format(self.domain),
+            query_params=filter_query,
         )
 
         records = []
@@ -127,7 +129,8 @@ def _update_record(self, identifier, rtype=None, name=None, content=None):
 
         for record_id in identifiers:
             self._put(
-                "/v1/domains/{0}/records/{1}".format(self.domain, identifier), record,
+                "/v1/domains/{0}/records/{1}".format(self.domain, identifier),
+                record,
             )
             LOGGER.debug("update_record: %s", record_id)
 

lexicon/providers/gehirn.py

@@ -188,7 +188,9 @@ def _delete_record(self, identifier=None, rtype=None, name=None, content=None):
             if "." not in identifier:
                 # delete entire record
                 path = "/zones/{}/versions/{}/records/{}".format(
-                    self.domain_id, self.version_id, identifier,
+                    self.domain_id,
+                    self.version_id,
+                    identifier,
                 )
                 self._delete(path)
                 LOGGER.debug("delete_record: %s", True)
@@ -212,7 +214,9 @@ def _delete_record(self, identifier=None, rtype=None, name=None, content=None):
                 if not record["records"]:
                     # delete entire record
                     path = "/zones/{}/versions/{}/records/{}".format(
-                        self.domain_id, self.version_id, record["id"],
+                        self.domain_id,
+                        self.version_id,
+                        record["id"],
                     )
                     self._delete(path)
                 else:
@@ -240,7 +244,9 @@ def _delete_record(self, identifier=None, rtype=None, name=None, content=None):
                     continue
 
             path = "/zones/{}/versions/{}/records/{}".format(
-                self.domain_id, self.version_id, a_record["id"],
+                self.domain_id,
+                self.version_id,
+                a_record["id"],
             )
             self._delete(path)
 
@@ -285,12 +291,17 @@ def _update_internal_record(self, record):
         if record.get("id"):
             # PUT
             path = "/zones/{}/versions/{}/records/{}".format(
-                self.domain_id, self.version_id, record["id"],
+                self.domain_id,
+                self.version_id,
+                record["id"],
             )
             return self._put(path, record)
 
         # POST
-        path = "/zones/{}/versions/{}/records".format(self.domain_id, self.version_id,)
+        path = "/zones/{}/versions/{}/records".format(
+            self.domain_id,
+            self.version_id,
+        )
         return self._post(path, record)
 
     def _build_content(self, rtype, record):

lexicon/providers/henet.py

@@ -24,13 +24,14 @@ def provider_parser(subparser):
         "--auth-username", help="specify username for authentication"
     )
     subparser.add_argument(
-        "--auth-password", help="specify password for authentication",
+        "--auth-password",
+        help="specify password for authentication",
     )
 
 
 class Provider(BaseProvider):
     """
-        he.net provider
+    he.net provider
     """
 
     def __init__(self, config):

lexicon/providers/infoblox.py

@@ -142,7 +142,10 @@ def _create_record_internal(self, rtype, name, content):
         else:
             raise Exception("Name not specified, no FQDN could be build")
         if rtype.upper() in IB_TYPE2CONTENT:
-            uri = "{0}{1}".format(self.api_endpoint, IB_TYPE2CONTENT[rtype.upper()][1],)
+            uri = "{0}{1}".format(
+                self.api_endpoint,
+                IB_TYPE2CONTENT[rtype.upper()][1],
+            )
             payload = self._generate_payload(rtype, name, content)
             try:
                 response = self.session.post(uri, data=json.dumps(payload))
@@ -285,7 +288,10 @@ def _delete_record(self, identifier=None, rtype=None, name=None, content=None):
 
     def _delete_record_internal(self, identifier=None):
         if identifier:
-            uri = "{0}{1}".format(self.api_endpoint, identifier,)
+            uri = "{0}{1}".format(
+                self.api_endpoint,
+                identifier,
+            )
             try:
                 response = self.session.delete(uri)
             except BaseException:

lexicon/providers/namecheap.py

@@ -262,7 +262,7 @@ def _delete_record(self, identifier=None, rtype=None, name=None, content=None):
         return True
 
     def _convert_to_namecheap(self, record):
-        """ converts from lexicon format record to namecheap format record,
+        """converts from lexicon format record to namecheap format record,
         suitable to sending through the api to namecheap"""
 
         processed_record = {}
@@ -292,8 +292,7 @@ def _convert_to_namecheap(self, record):
         return processed_record
 
     def _convert_to_lexicon(self, record):
-        """ converts from namecheap raw record format to lexicon format record
-        """
+        """converts from namecheap raw record format to lexicon format record"""
 
         name = record["Name"]
 

lexicon/tests/providers/test_directadmin.py

@@ -25,7 +25,7 @@ def _filter_query_parameters(self):
 
     def _filter_response(self, response):
         """See `IntegrationTests._filter_response` for more information on how
-           to filter the provider response."""
+        to filter the provider response."""
         return response
 
     def _test_parameters_overrides(self):

lexicon/tests/test_client.py

@@ -31,7 +31,7 @@ def test_client_basic_init():
 
 
 def test_client_legacy_init(monkeypatch):
-    monkeypatch.setenv('LEXICON_FAKEPROVIDER_AUTH_KEY', 'MY_KEY')
+    monkeypatch.setenv("LEXICON_FAKEPROVIDER_AUTH_KEY", "MY_KEY")
     options = {
         "provider_name": "fakeprovider",
         "action": "list",
@@ -45,7 +45,7 @@ def test_client_legacy_init(monkeypatch):
     assert client.action == options["action"]
     assert client.config.resolve("lexicon:domain") == options["domain"]
     assert client.config.resolve("lexicon:type") == options["type"]
-    assert client.config.resolve("lexicon:fakeprovider:auth_key") == 'MY_KEY'
+    assert client.config.resolve("lexicon:fakeprovider:auth_key") == "MY_KEY"
 
 
 def test_client_init_when_domain_includes_subdomain_should_strip():