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

Reference files using their ids or hashes instead of filenames. #112

Open
warownia1 opened this issue Aug 12, 2021 · 1 comment
Open

Reference files using their ids or hashes instead of filenames. #112

warownia1 opened this issue Aug 12, 2021 · 1 comment
Labels

Comments

@warownia1
Copy link
Collaborator

In the current implementation, result files are referenced by their paths rather than by random identifiers or, ideally, ids given in the service configuration file. The current approach is dictated by the file hosting issue #41 as for the proxy server to be able to find the file, its id must be the same as its path. This might cause potential problems with the REST API when file path is mixed with path parameters. Additionally, using predictable file ids such as <job_id>/<result_id>_<hash> would significantly help with automated job chaining. Managing files by slivka application and using X-Sendfile headers would be helpful, but break the compatibility with many proxy servers that does not support this header.

@warownia1 warownia1 added improvement help wanted feature All additional feature requests pri:med labels Aug 12, 2021
@warownia1 warownia1 changed the title Reference files using their ids. Reference files using their ids or hashes. Aug 23, 2021
@warownia1 warownia1 changed the title Reference files using their ids or hashes. Reference files using their ids or hashes instead of filenames. Aug 23, 2021
@warownia1 warownia1 added the v0.8.5 Featues and bugs that will go into the 0.8.5 release label Oct 27, 2021
@warownia1
Copy link
Collaborator Author

Related issue #41

@warownia1 warownia1 removed the v0.8.5 Featues and bugs that will go into the 0.8.5 release label Apr 9, 2024
@warownia1 warownia1 moved this to Todo in Slivka 0.8 release Apr 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Todo
Development

No branches or pull requests

1 participant