Added parameter to config to store certs

Author: mattmanley

maproulette/api/configuration.py

@@ -1,15 +1,37 @@
 """This module contains the basic configuration object that is used to communicate with the MapRoulette API."""
 
-DEFAULT_URL = 'maproulette.org'
+DEFAULT_HOSTNAME = 'maproulette.org'
 DEFAULT_PROTOCOL = 'https'
 DEFAULT_API_VERSION = '/api/v2'
 
 
 class Configuration:
     """Class for storing the configuration of the MapRoulette server"""
-    def __init__(self, url=DEFAULT_URL, protocol=DEFAULT_PROTOCOL, api_version=DEFAULT_API_VERSION, api_key=None):
-        self.url = f"{protocol}://{url}{api_version}"
-        self.base_url = f"{protocol}://{url}"
+    def __init__(self, hostname=DEFAULT_HOSTNAME, protocol=DEFAULT_PROTOCOL, api_version=DEFAULT_API_VERSION,
+                 api_key=None, certs=None, verify=True):
+        """Create a new configuration object to connect to the MapRoulette server.
+
+        :param hostname: Optional parameter to specify the hostname of the MapRoulette instance being addressed. Do not
+        include the protocol (https, http). Default value is 'maproulette.org'.
+        :type hostname: str
+        :param protocol: Optional parameter to specify the protocol to use for the connection to the MapRoulette
+        instance being addressed such as https or http. Default value is 'https'.
+        :type protocol: str
+        :param api_version: Optional parameter to specify the API version to use. The default is '/api/v2'.
+        :type api_version: str
+        :param api_key: Optional parameter to pass the user-specific API key. This key is necessary for some actions.
+        :type api_key: str
+        :param certs: Optional parameter to pass the client-side certificate and key if necessary to make connection
+        with the MapRoulette instance being addressed.
+        :type certs: tuple
+        :param verify: Optional parameter to specify whether to verify SSL certificates for HTTPS requests. Default is
+        True.
+        :type verify: bool
+        """
+        self.api_url = f"{protocol}://{hostname}{api_version}"
+        self.base_url = f"{protocol}://{hostname}"
+        self.protocol = protocol
         self.headers = dict()
         self.headers['apikey'] = api_key
-        self.protocol = protocol
+        self.certs = certs
+        self.verify = verify

maproulette/api/maproulette_server.py

@@ -10,13 +10,16 @@
 class MapRouletteServer:
     """Class that holds the basic requests that can be made to the MapRoulette API."""
     def __init__(self, configuration):
-        self.url = configuration.url
+        self.url = configuration.api_url
         self.base_url = configuration.base_url
         self.headers = configuration.headers
+        self.certs = configuration.certs
+        self.verify = configuration.verify
         if self.__check_health():
             self.session = requests.Session()
             self.session.headers = self.headers
-            self.session.verify = False
+            self.session.verify = self.verify
+            self.session.cert = self.certs
 
     def __check_health(self, retries=3, delay=5):
         """Checks health of connection to host by pinging the URL set in the configuration
@@ -30,7 +33,8 @@ def __check_health(self, retries=3, delay=5):
                 response = requests.get(
                     self.base_url + '/ping',
                     headers=self.headers,
-                    verify=False
+                    verify=self.verify,
+                    cert=self.certs
                 )
                 if not response.ok:
                     print(f"Unsuccessful connection. Retrying in {str(delay)} seconds")

maproulette/api/project.py

@@ -1,6 +1,5 @@
 """This module contains the methods that the user will use directly to interact with MapRoulette projects"""
 
-import json
 from maproulette.api.maproulette_server import MapRouletteServer
 from maproulette.models.project import ProjectModel