From 3d70dbd66e434d5ac58e8d018e566d3089223db3 Mon Sep 17 00:00:00 2001
From: Vitaly Chait <vitalyhait94@gmail.com>
Date: Tue, 24 Sep 2024 15:24:49 +0300
Subject: [PATCH] fix: url attribute type validation (#672)

---
 CHANGELOG.md                    |  3 +++
 influxdb_client/client/_base.py |  2 ++
 tests/test_InfluxDBClient.py    | 29 +++++++++++++++++++++++++++++
 3 files changed, 34 insertions(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index ebaccef3..6937e242 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,5 +1,8 @@
 ## 1.47.0 [unreleased]
 
+### Bug Fixes
+1. [#672](https://github.com/influxdata/influxdb-client-python/pull/672): Adding type validation to url attribute in client object
+
 ## 1.46.0 [2024-09-13]
 
 ### Bug Fixes
diff --git a/influxdb_client/client/_base.py b/influxdb_client/client/_base.py
index 8dcf75e9..d4f17901 100644
--- a/influxdb_client/client/_base.py
+++ b/influxdb_client/client/_base.py
@@ -53,6 +53,8 @@ def __init__(self, url, token, debug=None, timeout=10_000, enable_gzip=False, or
         self.default_tags = default_tags
 
         self.conf = _Configuration()
+        if not isinstance(self.url, str):
+            raise ValueError('"url" attribute is not str instance')
         if self.url.endswith("/"):
             self.conf.host = self.url[:-1]
         else:
diff --git a/tests/test_InfluxDBClient.py b/tests/test_InfluxDBClient.py
index 7fdf834f..228f391b 100644
--- a/tests/test_InfluxDBClient.py
+++ b/tests/test_InfluxDBClient.py
@@ -323,6 +323,35 @@ def test_version(self):
         version = self.client.version()
         self.assertTrue(len(version) > 0)
 
+    def test_url_attribute(self):
+        # Wrong URL attribute
+        wrong_types = [
+            None,
+            True, False,
+            123, 123.5,
+            dict({"url" : "http://localhost:8086"}),
+            list(["http://localhost:8086"]),
+            tuple(("http://localhost:8086"))
+        ]
+        correct_types = [
+            "http://localhost:8086"
+        ]
+        for url_type in wrong_types:
+            try:
+                client_not_running = InfluxDBClient(url=url_type, token="my-token", debug=True)
+                status = True
+            except ValueError as e:
+                status = False
+            self.assertFalse(status)
+        for url_type in correct_types:
+            try:
+                client_not_running = InfluxDBClient(url=url_type, token="my-token", debug=True)
+                status = True
+            except ValueError as e:
+                status = False
+            self.assertTrue(status)
+
+
     def test_build(self):
         build = self.client.build()
         self.assertEqual('oss', build.lower())