Bug fixes for the STAC export PRO feature
aaronxsu opened this issue · comments
exportAssetTypes
field should persist after signing the download URL.expiration
time field is also missing from the list and get endpoints, which is due to the same bug as the first item- Providing
exportAssetTypes: none
inPOST
request results in the imagery collection having no item attached
Both of the fields in item 1 and 2 are needed on the frontend. We should still support TMS-only exports (i.e. item 3 should work still)
The expiration... should be included 🤔
Unless it has a weird encoder? It doesn't look like we have a weird encoder 🤔
Oh, the build failed: http://jenkins.staging.rasterfoundry.com/job/Raster%20Foundry/job/raster-foundry/job/develop/1811/
@jisantuc fully agreed. BUT, look at here: https://github.com/raster-foundry/raster-foundry/blob/develop/app-backend/api/src/main/scala/stac/Routes.scala#L43-L73 and here: https://github.com/raster-foundry/raster-foundry/blob/develop/app-backend/datamodel/src/main/scala/StacExport.scala#L97-L111
The list and get endpoints both sign the download URL before returning. The case class WithSignedDownload
has neither expiration time nor export asset types
I see, I overlooked that in the initial work, nice catch 😎