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

Move downloadable zip delivery to the storage API #110

Open
booski opened this issue Apr 11, 2022 · 1 comment
Open

Move downloadable zip delivery to the storage API #110

booski opened this issue Apr 11, 2022 · 1 comment
Labels
enhancement New feature or request future work

Comments

@booski
Copy link

booski commented Apr 11, 2022

Keeping copies of presentations in zipped format on the app server is redundant and wastes space. The storage API should be extended to provide an endpoint to download a presentation as zip, but this requires a number of changes in the interaction between the app and storage servers:

  • All metadata for a presentation needs to be kept in the package file on the storage server (in addition to in the app server database). The storage server currently only keeps the 'minimal' package data; that being stream, poster and thumbnail data.
  • All metadata changes need to be propagated to the storage server as they happen.

Once the metadata is available and kept current on the storage server, presentations can be exposed as zipped packages at (tentatively) /presentation/zip/.

Tokens should be used for authenticating zip requests just as with other presentation files.

@booski booski added the enhancement New feature or request label Apr 11, 2022
@orkit
Copy link
Contributor

orkit commented Apr 19, 2022

I do not think this is really necessary. The risk is that there is redundant package information on both the play app and the play store. Currently, a download zip is created on demand with the current metadata and embedded in the current player. If it is a storage problem, a solution is not to save the zip file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request future work
Projects
None yet
Development

No branches or pull requests

2 participants