Merge pull request #902 from hubmapconsortium/Derek-Furst/support-public-version

updated comment block for get_entity_accessibility to reflect changes

Author: yuanzhou

src/app.py

@@ -588,22 +588,14 @@ class ReindexPriorityLevelEnum(Enum):
 def get_accessible_data_directories():
     dataset_helper = DatasetHelper()
 
-    # If not token is provided or an invalid token is provided, return a 401 error.
-    if request.headers.get('Authorization') is None:
-        unauthorized_error('A valid token must be provided.')
-
     # If an invalid token provided, we need to tell the client with a 401 error, rather
     # than a 500 error later if the token is not good.
     _validate_token_if_auth_header_exists(request)
 
-    # Get user token from Authorization header
-    # Get the user token from Authorization header
-    user_token = auth_helper_instance.getAuthorizationTokens(request.headers)
-
     # Get user group information which will be used to determine accessibility on
     # a per-entity basis.
     user_data_access_level = auth_helper_instance.getUserDataAccessLevel(request)
-
+    user_data_access_level['group_membership_ids'] = []
     if not request.is_json:
         bad_request_error("A json body and appropriate Content-Type header are required.")
     json_payload = request.get_json()

src/dataset_helper_object.py

@@ -121,15 +121,15 @@ def verify_dataset_title_info(self, dataset_uuid: str, user_token: str) -> array
 
         return rslt
 
-    # entity_id - UUID or HM_ID
-    # user_token - The authorization token for the user, which is used to generate an appropriate
-    #              description of the user's access to the entity.
+    # neo4j_driver - The driver instance for neo4j
+    # json_payload - A list of ids (HM_ID or UUID)
     # user_data_access_level - Data access level information for the user, notably including
     #                          Globus Group membership information.
     #
-    # Returns a JSON Object containing accessibility information for the entity.
-    #
-    def get_entity_accessibility(self, neo4j_driver, json_payload, user_data_access_level: dict = None) -> dict:
+    # Returns a Dict of Dicts where each of the dicts inside is keyed by its original id given
+    # in the json_payload and contains information about the accessibility of that directory 
+    # including its globus url. 
+    def get_entity_accessibility(self, neo4j_driver, json_payload, user_data_access_level) -> dict:
         supported_entity_type_list = ['Dataset', 'Upload']
         accessibility_dicts = {}