raster-foundry / raster-foundry

The Raster Foundry web application.

Home Page:https://www.rasterfoundry.com

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

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 in POST 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)

@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 😎