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

S3 asset upload for deploy command #39

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Conversation

detjensrobert
Copy link
Contributor

@detjensrobert detjensrobert commented Feb 8, 2025

Implements the asset file upload portion of the deploy command.

Asset files are uploaded to <the bucket>/assets/${CHAL/SLUG}/${FILENAME} (e.g. /assets/misc/foo/things.zip)

Depends on #38 and includes its commits.

@detjensrobert detjensrobert self-assigned this Feb 8, 2025
These were causing nested tokio runtime panics when calling `build()` from
`deploy()`. All of the callers are async functions anyway, so these can be
true async functions now.

Signed-off-by: Robert Detjens <[email protected]>
This is cleaner than needing to wrap the whole iterator in a call to
futures::try_join_all().

Signed-off-by: Robert Detjens <[email protected]>
Signed-off-by: Robert Detjens <[email protected]>
This makes things cleaner than a big `async move` block

Signed-off-by: Robert Detjens <[email protected]>
Signed-off-by: Robert Detjens <[email protected]>
@detjensrobert detjensrobert force-pushed the dr/deploy-upload branch 3 times, most recently from 5a7f594 to 87915dc Compare February 8, 2025 21:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant