make sure user_email field doesn't override contacts_wrest when it's an analyst list only

services/capability/capability/views/capability_version.py

@@ -25,7 +25,6 @@ import json
 import logging
 import subprocess
 
-import requests
 from pycapo import CapoConfig
 from pyramid.httpexceptions import HTTPBadRequest, HTTPNotFound, HTTPPreconditionFailed
 from pyramid.request import Request
@@ -394,9 +393,12 @@ def get_user_email(request: Request) -> Response:
         if not version.parameters:
             not_found_msg = f"No parameters found for version {version_id} of request with ID {capability_request_id}. Couldn't find user_email information."
             return HTTPNotFound(detail=not_found_msg)
-        elif "user_email" in version.parameters and version.parameters["user_email"]:
+        elif ("user_email" in version.parameters and version.parameters["user_email"]
+              and "workspaces-analysts" not in version.parameters["user_email"]):
+            # exclude the case of follow-on requests when the default user_email is the analyst list,
+            # otherwise send to specified address
             return Response(status_int=http.HTTPStatus.OK, json_body={"resp": f"{version.parameters['user_email']}"})
-        # If user_email not populated, search for the PI/CoI emails with the contacts_wrester
+        # If user_email not populated (or is an analyst list), search for the PI/CoI emails with the contacts_wrester
         elif "product_locator" in version.parameters and version.parameters["product_locator"]:
             process_call = ["aat_wrest", "-sc", version.parameters["product_locator"]]
         elif "sdmId" in version.parameters and version.parameters["sdmId"]: