fix: python

Author: shortcuts

clients/algoliasearch-client-python/algoliasearch/http/transporter_sync.py

@@ -66,6 +66,8 @@ def request(
             url = self.build_url(host, path)
             proxies = self.get_proxies(url)
 
+            print(url)
+
             req = Request(
                 method=verb,
                 url=url,

playground/python/app/search.py

@@ -11,26 +11,26 @@
 def main():
     print("SearchClient version", __version__)
 
-    client = SearchClientSync(
-        environ.get("ALGOLIA_APPLICATION_ID"), environ.get("ALGOLIA_ADMIN_KEY")
-    )
-    client.add_user_agent("playground")
-    client.add_user_agent("bar", "baz")
-
-    print("user_agent", client._config._user_agent.get())
-    print("client initialized", client)
-
-    try:
-        resp = client.search_synonyms("foo")
-        print(resp)
-        client.browse_synonyms("foo", lambda _resp: print(_resp))
-    finally:
-        client.close()
-
-        print("client closed")
-
-    print("with transformations")
-
+    # client = SearchClientSync(
+    #     environ.get("ALGOLIA_APPLICATION_ID"), environ.get("ALGOLIA_ADMIN_KEY")
+    # )
+    # client.add_user_agent("playground")
+    # client.add_user_agent("bar", "baz")
+    #
+    # print("user_agent", client._config._user_agent.get())
+    # print("client initialized", client)
+    #
+    # try:
+    #     resp = client.search_synonyms("foo")
+    #     print(resp)
+    #     client.browse_synonyms("foo", lambda _resp: print(_resp))
+    # finally:
+    #     client.close()
+    #
+    #     print("client closed")
+    #
+    # print("with transformations")
+    #
     config = SearchConfig(
         environ.get("ALGOLIA_APPLICATION_ID"), environ.get("ALGOLIA_ADMIN_KEY")
     )
@@ -46,10 +46,12 @@ def main():
     print("user_agent", client._config._user_agent.get())
 
     try:
-        resp = client.save_objects_with_transformation(
-            "foo", [{"objectID": "bar"}], wait_for_tasks=True
+        resp = client.replace_all_objects_with_transformation(
+            "boyd", [{"objectID": "bar"}]
         )
         print(resp)
+    except Exception as e:
+        print(e)
     finally:
         client.close()
 

playground/python/poetry.lock

@@ -86,18 +86,27 @@ class {{classname}}{{#isSyncClient}}Sync{{/isSyncClient}}:
         if transporter is None:
             transporter = Transporter{{#isSyncClient}}Sync{{/isSyncClient}}(config)
 
-        client = {{classname}}{{#isSyncClient}}Sync{{/isSyncClient}}(app_id=config.app_id, api_key=config.api_key, {{#hasRegionalHost}}region=config.region, {{/hasRegionalHost}}transporter=transporter, config=config)
-
         {{#isSearchClient}}
+        _ingestion_transporter: Optional[IngestionClient{{#isSyncClient}}Sync{{/isSyncClient}}] = None
+
         if config.region is not None:
             ingestion_config = IngestionConfig(config.app_id, config.api_key, config.region)
 
             if config.hosts is not None:
                 ingestion_config.hosts = config.hosts
 
-            client._ingestion_transporter = IngestionClient{{#isSyncClient}}Sync{{/isSyncClient}}.create_with_config(ingestion_config)
+            _ingestion_transporter = IngestionClient{{#isSyncClient}}Sync{{/isSyncClient}}.create_with_config(ingestion_config)
         {{/isSearchClient}}
 
+        config.set_default_hosts()
+
+        client = {{classname}}{{#isSyncClient}}Sync{{/isSyncClient}}(app_id=config.app_id, api_key=config.api_key, {{#hasRegionalHost}}region=config.region, {{/hasRegionalHost}}transporter=transporter, config=config)
+
+        {{#isSearchClient}}
+        if _ingestion_transporter is not None:
+            client._ingestion_transporter = _ingestion_transporter
+        {{/isSearchClient}}
+        
         return client
 
     {{^isSyncClient}}

templates/python/api.mustache

@@ -46,14 +46,30 @@ class {{#lambda.pascalcase}}{{client}}{{/lambda.pascalcase}}Config(BaseConfig):
         if https_proxy is not None:
             self.proxies["https"] = https_proxy
 
+
         {{#hasRegionalHost}}
         self.region = region
+        {{/hasRegionalHost}}
+
+    {{#isSearchClient}}
+        self.region = None
+
+    def set_transformation_region(self, region: str = ""):
+        "This method is required to be called with the appropriate region of your Algolia application if you wish to leverage the *_with_transformation methods."
+        self.region = region
+    {{/isSearchClient}}
+
+    def set_default_hosts(self):
+        if self.hosts is not None:
+            return
+
+        {{#hasRegionalHost}}
         _regions = [{{#allowedRegions}}'{{.}}'{{^-last}},{{/-last}}{{/allowedRegions}}]
 
-        if {{^fallbackToAliasHost}}not region or {{/fallbackToAliasHost}}(region is not None and region not in _regions):
+        if {{^fallbackToAliasHost}}not self.region or {{/fallbackToAliasHost}}(self.region is not None and self.region not in _regions):
             raise ValueError(f"`region` {{^fallbackToAliasHost}}is required and {{/fallbackToAliasHost}}must be one of the following: {', '.join(_regions)}")
 
-        self.hosts = HostsCollection([Host({{#fallbackToAliasHost}}"{{{hostWithFallback}}}" if region is None else {{/fallbackToAliasHost}} "{{{regionalHost}}}".replace("{region}", region or ""))])
+        self.hosts = HostsCollection([Host({{#fallbackToAliasHost}}"{{{hostWithFallback}}}" if self.region is None else {{/fallbackToAliasHost}} "{{{regionalHost}}}".replace("{region}", self.region or ""))])
         {{/hasRegionalHost}}
 
         {{^hasRegionalHost}}
@@ -76,12 +92,4 @@ class {{#lambda.pascalcase}}{{client}}{{/lambda.pascalcase}}Config(BaseConfig):
             reorder_hosts=True,
             {{/hostWithAppID}}
         )
-        {{/hasRegionalHost}}
-
-    {{#isSearchClient}}
-        self.region = None
-
-    def set_transformation_region(self, region: str = ""):
-        "This method is required to be called with the appropriate region of your Algolia application if you wish to leverage the *_with_transformation methods."
-        self.region = region
-    {{/isSearchClient}}
+        {{/hasRegionalHost}}