W-16337942: Allow existing functions to be updated to public without being redefined

Author: jmoens

tabpy/tabpy_server/handlers/endpoint_handler.py

@@ -95,6 +95,7 @@ def delete(self, name):
             return
 
         self.logger.log(logging.DEBUG, f"Processing DELETE for /endpoints/{name}")
+        self.logger.log(logging.INFO, f"Processing DELETE for /endpoints/{name}")
 
         try:
             endpoints = self.tabpy_state.get_endpoints(name)

tabpy/tabpy_server/management/state.py

@@ -428,6 +428,36 @@ def delete_endpoint(self, name):
             logger.error(f"Unable to delete endpoint {e}")
             raise ValueError(f"Unable to delete endpoint: {e}")
 
+
+    @state_lock
+    def make_endpoint_public(self, name):
+        """
+        Updates an existing endpoint on the TabPy Server to be public
+
+        Parameters
+        ----------
+        name : str
+            The name of the endpoint to be updated.
+
+        Returns
+        -------
+        updated endpoint object
+
+        """
+        if not name or name == "":
+            raise ValueError("Name of the endpoint must be a valid string.")
+        endpoints = self.get_endpoints()
+        if name not in endpoints:
+            raise ValueError(f"Endpoint {name} does not exist.")
+
+        endpoint_to_update = endpoints[name]
+
+        
+        self.update_endpoint(
+            name,
+            isPublic=True
+        )
+
     @property
     def name(self):
         """

tabpy/tabpy_tools/client.py

@@ -264,6 +264,16 @@ def remove(self, name):
             Endpoint name to remove'''
         self._service.remove_endpoint(name)
 
+    def make_public(self, name):
+        '''Makes an existing endpoint public.
+
+        Parameters
+        ----------
+        name : str
+            Endpoint name to make public'''
+
+        self._service.make_public(name)
+
     def _gen_endpoint(self, name, obj, description, version=1, schema=None, isPublic=False):
         """Generates an endpoint dict.
 

tabpy/tabpy_tools/rest.py

@@ -117,6 +117,7 @@ def PUT(self, url, data, timeout=None):
         data = self._encode_request(data)
 
         logger.info(f"PUT {url} with {data}")
+        logger.log(logging.INFO, f"PUT {url} with {data}")
 
         response = self.session.put(
             url,

tabpy/tabpy_tools/rest_client.py

@@ -216,6 +216,18 @@ def set_endpoint(self, endpoint):
         """
         return self.service_client.PUT("endpoints/" + endpoint.name, endpoint.to_json())
 
+    def make_public(self, endpoint_name):
+        """Updates an existing endpoint to be public.
+
+        Parameters
+        ----------
+
+        endpoint_name : str
+
+            The name of the endpoint.
+        """
+        self.service_client.PUT("endpoints/" + endpoint_name)
+
     def remove_endpoint(self, endpoint_name):
         """Deletes an endpoint through the management API.