Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Parameterize maap-py granule.getDownloadUrl() #848

Closed
bsatoriu opened this issue Oct 25, 2023 · 2 comments
Closed

Parameterize maap-py granule.getDownloadUrl() #848

bsatoriu opened this issue Oct 25, 2023 · 2 comments
Assignees
Labels
API MAAP API Subsystem
Milestone

Comments

@bsatoriu
Copy link
Collaborator

maap-py's granule.getDownloadUrl() method retrieves the s3 uri for the granule. In many cases, the http url for the granule is preferred for security reasons. Add a parameter to this method to specifiy the data source, and default to https.

@bsatoriu bsatoriu added the API MAAP API Subsystem label Oct 25, 2023
@bsatoriu bsatoriu self-assigned this Oct 25, 2023
@bsatoriu bsatoriu added this to the 3.1.4 milestone Nov 9, 2023
@bsatoriu
Copy link
Collaborator Author

bsatoriu commented Dec 7, 2023

Testing notes

results = maap.searchGranule(page_num="1", concept_id="C1214470488-ASF", sort_key="-start_date", limit=1)

url = results[0].getHttpUrl()
# url should start with "http"

url = results[0].getS3Url()
# url should start with "s3"

@marjo-luc
Copy link
Member

Tested successfully in OPS.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API MAAP API Subsystem
Projects
None yet
Development

No branches or pull requests

2 participants