Remove debugging statements

ohsu-comp-bio · Dec 15, 2023 · 3839a4d · 3839a4d
1 parent 6703a91
commit 3839a4d
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 6 deletions.
diff --git a/fence/blueprints/data/indexd.py b/fence/blueprints/data/indexd.py
@@ -1150,7 +1150,6 @@ def complete_multipart_upload(self, uploadId, parts, expires_in):
 
     def delete(self, bucket, file_id):
         try:
-            print(f"DEBUG boto: {flask.current_app.boto}")
             return flask.current_app.boto.delete_data_file(bucket, file_id)
         except Exception as e:
             logger.error(e)

diff --git a/fence/blueprints/data/multipart_upload.py b/fence/blueprints/data/multipart_upload.py
@@ -159,10 +159,9 @@ def generate_presigned_url_for_uploading_part(
     additional_signed_qs = {"partNumber": str(partNumber), "uploadId": uploadId}
 
     try:
-        presigned_url = generate_aws_presigned_url(
+        return generate_aws_presigned_url(
             url, "PUT", credentials, "s3", region, expires, additional_signed_qs
         )
-        return presigned_url
     except Exception as e:
         raise InternalError(
             "Can not generate presigned url for part number {} of key {}. Detail {}".format(

diff --git a/fence/resources/aws/boto_manager.py b/fence/resources/aws/boto_manager.py
@@ -30,9 +30,8 @@ def create_s3_clients(self, config, buckets):
             'default': client('s3', **config)
         }
         for bucket in buckets:
-            print(f"DEBUG bucket: {bucket}")
-            if buckets[bucket]['endpoint_url'] is not None:
-                print(f"DEBUG endpoint_url: {endpoint_url}")
+            if 'endpoint_url' in buckets[bucket]:
+                endpoint_url = buckets[bucket]['endpoint_url']
                 endpoint_url = buckets[bucket]['endpoint_url']
                 s3_clients[bucket] = client('s3', **config, endpoint_url=endpoint_url)
         return s3_clients