Skip to content

Commit 55812c5

Browse files
committed
fix: need to pass token to actual call as well
1 parent d73133c commit 55812c5

File tree

3 files changed

+8
-7
lines changed

3 files changed

+8
-7
lines changed

cumulus_lambda_functions/uds_api/dapa/granules_dapa_query_es.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -154,7 +154,7 @@ def get_single_granule(self, granule_id):
154154
self.__restructure_each_granule_result(each_granules_query_result_stripped)
155155
return each_granules_query_result_stripped
156156

157-
def delete_facade(self, current_url: URL):
157+
def delete_facade(self, current_url: URL, bearer_token: str):
158158
actual_path = current_url.path
159159
actual_path = actual_path if actual_path.endswith('/') else f'{actual_path}/'
160160
actual_path = f'{actual_path}actual'
@@ -165,12 +165,12 @@ def delete_facade(self, current_url: URL):
165165
'path': actual_path,
166166
'httpMethod': 'DELETE',
167167
'headers': {
168-
'Accept': '*/*', 'Accept-Encoding': 'gzip, deflate', 'Authorization': 'Bearer xxx',
168+
'Accept': '*/*', 'Accept-Encoding': 'gzip, deflate', 'Authorization': bearer_token,
169169
'Host': current_url.hostname, 'User-Agent': 'python-requests/2.28.2',
170170
'X-Amzn-Trace-Id': 'Root=1-64a66e90-6fa8b7a64449014639d4f5b4', 'X-Forwarded-For': '44.236.15.58',
171171
'X-Forwarded-Port': '443', 'X-Forwarded-Proto': 'https'},
172172
'multiValueHeaders': {
173-
'Accept': ['*/*'], 'Accept-Encoding': ['gzip, deflate'], 'Authorization': ['Bearer xxx'],
173+
'Accept': ['*/*'], 'Accept-Encoding': ['gzip, deflate'], 'Authorization': [bearer_token],
174174
'Host': [current_url.hostname], 'User-Agent': ['python-requests/2.28.2'],
175175
'X-Amzn-Trace-Id': ['Root=1-64a66e90-6fa8b7a64449014639d4f5b4'],
176176
'X-Forwarded-For': ['127.0.0.1'], 'X-Forwarded-Port': ['443'], 'X-Forwarded-Proto': ['https']

cumulus_lambda_functions/uds_api/granules_api.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -308,7 +308,7 @@ async def delete_single_granule_dapa_facade(request: Request, collection_id: str
308308
try:
309309
LOGGER.debug(f'deleting granule: {granule_id}')
310310
granules_dapa_query = GranulesDapaQueryEs(collection_id, -1, -1, None, None, None, '')
311-
delete_prep_result = granules_dapa_query.delete_facade(request.url)
311+
delete_prep_result = granules_dapa_query.delete_facade(request.url, request.headers.get('Authorization', ''))
312312
except Exception as e:
313313
LOGGER.exception('failed during delete_single_granule_dapa')
314314
raise HTTPException(status_code=500, detail=str(e))

tests/integration_tests/test_granules_deletion.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
import base64
22
import json
33
import os
4+
from time import sleep
45
from unittest import TestCase
56

67
import requests
@@ -32,7 +33,7 @@ def setUp(self) -> None:
3233

3334
self.tenant = 'UDS_LOCAL_TEST_3' # 'uds_local_test' # 'uds_sandbox'
3435
self.tenant_venue = 'DEV' # 'DEV1' # 'dev'
35-
self.collection_name = 'CCC-04' # 'uds_collection' # 'sbx_collection'
36+
self.collection_name = 'CCC-05' # 'uds_collection' # 'sbx_collection'
3637
self.collection_version = '08'.replace('.', '') # '2402011200'
3738
return
3839

@@ -79,10 +80,10 @@ def test_delete_all(self):
7980
query_result = requests.delete(url=post_url,
8081
headers=headers,
8182
)
82-
self.assertEqual(query_result.status_code, 200, f'wrong status code. {query_result.text}')
83+
self.assertEqual(query_result.status_code, 202, f'wrong status code. {query_result.text}')
8384
response_json = json.loads(query_result.text)
8485
print(json.dumps(response_json, indent=4))
85-
86+
sleep(30)
8687
post_url = f'{self.uds_url}collections/{collection_id}/items/' # MCP Dev
8788
query_result = requests.get(url=post_url, headers=headers,)
8889
self.assertEqual(query_result.status_code, 200, f'wrong status code. {query_result.text}')

0 commit comments

Comments
 (0)