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

Add signed zip download link to stac exports #5220

Merged
merged 9 commits into from
Oct 16, 2019
Merged

Conversation

Lknechtli
Copy link
Contributor

@Lknechtli Lknechtli commented Oct 11, 2019

Overview

Allow the user to download the entire catalog zipfile in an http link

Checklist

  • Description of PR is in an appropriate section of the changelog and grouped with similar changes if possible
  • Swagger specification updated
  • New tables and queries have appropriate indices added
  • Any content changes are properly templated using BUILDCONFIG.APP_NAME
  • Any new SQL strings have tests

Demo

catalog.zip

Testing Instructions

  • batch/assembly
  • Spin up servers and frontend. Grab a token.
  • POST below object to /api/stac:
{
	"name": "florence al test 1",
	"layerDefinitions": [
		{
			"projectId": "7e584c31-f5d1-4a02-9428-e83006642375",
			"layerId": "1a8c1632-fa91-4a62-b33e-3a87c2ebdf16"
		}
	],
	"taskStatuses": ["LABELED", "VALIDATED"]
}
  • Grab the above export record's ID
  • docker-compose build batch
  • ./scripts/console batch bash
  • java -cp /opt/raster-foundry/jars/batch-assembly.jar com.rasterfoundry.batch.Main write_stac_catalog <stac_export_id>
  • make a GET request to /api/stac with your credentials
  • Copy the downloadUrl field and paste it into your address bar to download.
  • verify that the contents of the zipfile are correct.

Closes #5206

@Lknechtli Lknechtli requested a review from aaronxsu October 11, 2019 15:07
@Lknechtli Lknechtli marked this pull request as ready for review October 11, 2019 15:07
Copy link
Member

@aaronxsu aaronxsu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works as described. Should be good to go after the suggested change.

app-backend/api/src/main/scala/stac/Routes.scala Outdated Show resolved Hide resolved
@Lknechtli Lknechtli merged commit 486496c into develop Oct 16, 2019
@Lknechtli Lknechtli deleted the feature/lk/stac-zip branch October 16, 2019 20:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Add Zipped STAC export + Direct Download Signed URL
2 participants