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

We ask for filename in the Content-Disposition but we don't do anything with it #13

Open
richard-jones opened this issue Jan 10, 2020 · 2 comments
Assignees
Labels

Comments

@richard-jones
Copy link
Contributor

During deposit, filename is requested in the Content-Disposition header, when sending a binary/package. We then don't place any onus on the server to do anything with this information, and we don't provide any mechanism for this information to be included in the status document.

Is filename even meaningful? It MAY affect the @id that's given to the file, but it may also not.

What should we do? Get rid of filename in Content-Disposition, keep it and just document it's usage, or make space for it in the status.link field?

@alexdutton
Copy link
Contributor

FWIW, invenio-sword uses the filename to set the file key (i.e. the filename), which is then used in constructing the @id.

AIUI, the spec effectively says a deposit is a set of identifiable but unstructured files, with neither a directory structure, or even filenames. This probably wants resolving, even if every implementation is going to pay heed to filenames anyway.

@richard-jones
Copy link
Contributor Author

This is kind of a duplicate of #9 - I've left it that the spec allows the client to specify it, and that the server SHOULD take it into account, with no requirement to do so. On update the server MAY do something with it but may not. So this is just a hint to the server that this is what you would like to call the file. I don't think I'm going to say any more about it in the spec, though @alexsdutton this may be something for the implementers guides?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants