nbforager 0.8.1__tar.gz → 0.8.2__tar.gz

{nbforager-0.8.1 → nbforager-0.8.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: nbforager
-Version: 0.8.1
+Version: 0.8.2
 Summary: Python package designed to assist in working with the Netbox REST API. The filter parameters are identical to those in the Web UI filter form. It replaces brief data with full information, and Netbox objects are represented as a recursive multidimensional dictionary.
 License: Apache-2.0
 Keywords: netbox
@@ -23,7 +23,7 @@ Requires-Dist: requests (>=2,<3)
 Requires-Dist: tabulate (>=0.9,<0.10)
 Requires-Dist: vhelpers (>=0.3)
 Project-URL: Bug Tracker, https://github.com/vladimirs-git/nbforager/issues
-Project-URL: Download URL, https://github.com/vladimirs-git/nbforager/archive/refs/tags/0.8.1.tar.gz
+Project-URL: Download URL, https://github.com/vladimirs-git/nbforager/archive/refs/tags/0.8.2.tar.gz
 Project-URL: homepage, https://github.com/vladimirs-git/nbforager
 Project-URL: repository, https://github.com/vladimirs-git/nbforager
 Description-Content-Type: text/x-rst

{nbforager-0.8.1 → nbforager-0.8.2}/nbforager/api/base_c.py

@@ -13,13 +13,13 @@ from operator import itemgetter
 from queue import Queue
 from threading import Thread
 from typing import Callable
-from urllib.parse import urlencode, ParseResult
+from urllib.parse import ParseResult
 
 import netports
 import requests
 from requests import Session, Response
 from requests.exceptions import ReadTimeout, ConnectionError as RequestsConnectionError
-from vhelpers import vdict, vlist, vparam
+from vhelpers import vlist, vparam
 
 from nbforager import helpers as h
 from nbforager.api import extended_get
@@ -293,7 +293,7 @@ class BaseC:
         params_d_["brief"] = 1
         params_d_["limit"] = 1
         params_l: LParam = vparam.from_dict(params_d_)
-        url = f"{self.url_base}{path}?{urlencode(params_l)}"
+        url = f"{self.url_base}{path}?{urllib.parse.urlencode(params_l)}"
         response: Response = self._retry_requests(url)
 
         count = 0
@@ -316,62 +316,43 @@ class BaseC:
 
         :return: Netbox objects. Update self _results.
         """
-        offset = 0
-        if offsets := params_d.get("offset"):
-            offset = int(offsets[0])
+        params_d = params_d.copy()
+        if "limit" not in params_d:
+            params_d["limit"] = [self.limit]
+        if "offset" not in params_d:
+            params_d["offset"] = [0]
 
-        max_limit: int = self._set_limit(params_d)
         params_l: LParam = vparam.from_dict(params_d)
+        url = f"{self.url_base}{path}?{urllib.parse.urlencode(params_l)}"
 
         results: LDAny = []
         while True:
-            params_i = [*params_l, ("offset", offset)]
-            url = f"{self.url_base}{path}?{urlencode(params_i)}"
             response: Response = self._retry_requests(url)
-            if response.ok:
-                html: str = response.content.decode("utf-8")
-                data: DAny = json.loads(html)
-                results_: LDAny = list(data["results"])
-                results.extend(results_)
-            else:
-                results_ = []
-
-            # stop requests if limit reached
-            if self.limit != len(results_):
+            if not response.ok:
                 break
-            if max_limit and max_limit <= len(results):
+
+            html: str = response.content.decode("utf-8")
+            data: DAny = json.loads(html)
+            results_: LDAny = list(data["results"])
+            results.extend(results_)
+
+            # retrieve next offset from Response
+            if url_next := str(data.get("next") or ""):
+                url = url_next
+            else:
                 break
 
-            # next iteration
+            # Sleep before the next iteration to reduce server load
             if self.interval:
                 time.sleep(self.interval)
-            offset += self.limit
 
         return results
 
-    def _set_limit(self, params_d: DList) -> int:
-        """Update limit valur in params_d based on limit and max_limit
-
-        :return: Max limit value, update params_d["limit"] value
-        """
-        limit = 0
-        if limit_ := list(vdict.pop(data=params_d, key="limit") or []):
-            limit = int(limit_[0])
-        if not limit:
-            limit = self.limit
-        max_limit = 0
-        if max_limit_ := list(vdict.pop(data=params_d, key="max_limit") or []):
-            max_limit = int(max_limit_[0])
-        if max_limit and max_limit < limit:
-            limit = max_limit
-        params_d["limit"] = [limit]
-        return max_limit
-
     def _query_data_thread(self, path: str, params_d: DAny) -> None:
-        """Retrieve data from the Netbox.
+        """Retrieve data from the Netbox and appends results to the internal results list.
 
-        If the number of items in the result exceeds the limit, iterate through the offset
-        in a loop mode.
+        If the params_d does not have `offset` and the received Response contains "next" URL,
+            the method will recurse to fetch the next set of data.
 
         :param path: Section of the URL that points to the model.
         :param params_d: Parameters to request from the Netbox.
@@ -379,12 +360,26 @@ class BaseC:
         :return: Netbox objects. Update self _results.
         """
         params_l: LParam = vparam.from_dict(params_d)
-        url = f"{self.url_base}{path}?{urlencode(params_l)}"
+        url = f"{self.url_base}{path}?{urllib.parse.urlencode(params_l)}"
+
         response: Response = self._retry_requests(url)
-        if response.ok:
-            html: str = response.content.decode("utf-8")
-            data: DAny = json.loads(html)
-            results_: LDAny = list(data["results"])
+        if not response.ok:
+            return
+
+        html: str = response.content.decode("utf-8")
+        data: DAny = json.loads(html)
+        results_: LDAny = list(data["results"])
+        self._results.extend(results_)
+
+        # threading mode manage offset in top level method
+        if "offset" in params_d:
+            return
+
+        # retrieve next offset from Response
+        if url_next := str(data.get("next") or ""):
+            url_o: ParseResult = urllib.parse.urlparse(url_next)
+            params_d_: DAny = urllib.parse.parse_qs(url_o.query)
+            results_ = self._query_loop(path=path, params_d=params_d_)
             self._results.extend(results_)
 
     def _query_pages_count(self, params_ld: LDList) -> LDAny:

{nbforager-0.8.1 → nbforager-0.8.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "nbforager"
-version = "0.8.1"
+version = "0.8.2"
 description = "Python package designed to assist in working with the Netbox REST API. The filter parameters are identical to those in the Web UI filter form. It replaces brief data with full information, and Netbox objects are represented as a recursive multidimensional dictionary."
 authors = ["Vladimirs Prusakovs <vladimir.prusakovs@gmail.com>"]
 readme = "README.rst"
@@ -19,7 +19,7 @@ classifiers = [
 homepage = "https://github.com/vladimirs-git/nbforager"
 repository = "https://github.com/vladimirs-git/nbforager"
 "Bug Tracker" = "https://github.com/vladimirs-git/nbforager/issues"
-"Download URL" = "https://github.com/vladimirs-git/nbforager/archive/refs/tags/0.8.1.tar.gz"
+"Download URL" = "https://github.com/vladimirs-git/nbforager/archive/refs/tags/0.8.2.tar.gz"
 
 [tool.poetry.dependencies]
 python = "^3.11"