fix: include checksum in event handler #52
+399
−193
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What I am changing
This PR updates how we handle finding the Sentinel-2 data checksum when using the "subscriptions" based "link fetcher". When we attempt to download granules that we discover through ESA's Subscriptions API we can begin downloading so quickly that ESA doesn't have the checksum metadata available for the granule. This is OK because the subscription API payload includes the checksum, and this can save us an API request to ESA 🎉.
There are two complications to this that are handled,
For more background see this writeup of the trial I ran over the last weekend,
https://github.com/NASA-IMPACT/hls_development/issues/300#issuecomment-2648409964
How I did it
requests
to fix a CVE that Dependabot flaggedHow you can test it
I updated the unit tests to include,