Fix SSL verify (#140)

docs/advanced/session.rst

@@ -10,7 +10,6 @@ A few examples are provided below:
 * :ref:`SSL Verification`
 * :ref:`Timeouts`
 
-
 Headers
 -------
 
@@ -31,11 +30,9 @@ The example below shows how to update a Token if it has been cycled.
     # Update Session object with new header
     nautobot.http_session.headers["Authorization"] = new_token
 
-
 SSL Verification
 ----------------
 
-Handling SSL Verification is documented `here <https://requests.readthedocs.io/en/stable/user/advanced/#ssl-cert-verification>`_.
 The below example shows how to disable SSL verification.
 
 .. code-block:: python
@@ -44,10 +41,9 @@ The below example shows how to disable SSL verification.
     from pynautobot import api
     nautobot = api(
         url='https://localhost:8000',
-        token=os.environ["NAUTOBOT_TOKEN"]
+        token=os.environ["NAUTOBOT_TOKEN"],
+        verify=False
     )
-    nautobot.http_session.verify = False
-
 
 Timeouts
 --------

pynautobot/core/api.py

@@ -61,6 +61,7 @@ class Api(object):
         for all requests.
     :param int,optional retries: Number of retries, for HTTP codes 429, 500, 502, 503, 504,
         this client will try before dropping.
+    :param bool,optional verify: SSL cert verification.
     :raises AttributeError: If app doesn't exist.
     :Examples:
 
@@ -80,12 +81,14 @@ def __init__(
         max_workers=4,
         api_version=None,
         retries=0,
+        verify=True,
     ):
         base_url = "{}/api".format(url if url[-1] != "/" else url[:-1])
         self.token = token
         self.headers = {"Authorization": f"Token {self.token}"}
         self.base_url = base_url
         self.http_session = requests.Session()
+        self.http_session.verify = verify
         if retries:
             _adapter = HTTPAdapter(
                 max_retries=Retry(

tests/test_api.py

@@ -40,6 +40,18 @@ def test_sanitize_url(self, *_):
         self.assertTrue(api)
         self.assertEqual(api.base_url, "http://localhost:8000/api")
 
+    @patch("pynautobot.api.version", "1.999")
+    def test_verify_true(self, *_):
+        api = pynautobot.api("http://localhost:8000/", **def_kwargs)
+        self.assertTrue(api)
+        self.assertTrue(api.http_session.verify)
+
+    @patch("pynautobot.api.version", "1.999")
+    def test_verify_false(self, *_):
+        api = pynautobot.api("http://localhost:8000/", verify=False, **def_kwargs)
+        self.assertTrue(api)
+        self.assertFalse(api.http_session.verify)
+
 
 class ApiVersionTestCase(unittest.TestCase):
     class ResponseHeadersWithVersion: