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

Use premis hasOriginalName (and hasSize) predicates on fedora:binary resources #179

Merged
merged 1 commit into from Dec 4, 2013

Conversation

cbeer
Copy link
Contributor

@cbeer cbeer commented Dec 2, 2013

Expose the premis:hasOriginalName propertyfrom the HTTP API using
the Content-Disposition ("filename") header for both fcr:content
and a POST with a binary payload.

Expose originalFileName throughout the kernel's Datastream API.

Fixes https://www.pivotaltracker.com/story/show/61500736

@ajs6f
Copy link
Contributor

ajs6f commented Dec 2, 2013

@cbeer : Just to be sure, does this play safely with federation (when the original filename may not have been recorded or even exist)?

@cbeer
Copy link
Contributor Author

cbeer commented Dec 2, 2013

@ajs6f yes? premis:hasOriginalName is optional, if it's not there, we don't set the property.

@ajs6f
Copy link
Contributor

ajs6f commented Dec 2, 2013

Cool. That works for me.

resources

Expose the premis:hasOriginalName propertyfrom the HTTP API using
the Content-Disposition ("filename") header for both fcr:content
and a POST with a binary payload.

Expose originalFileName throughout the kernel's Datastream API.

Use Content-Disposition headers when retrieving content to hint
the original file name, creation and modification times.

Fixes https://www.pivotaltracker.com/story/show/61500736
awoods pushed a commit that referenced this pull request Dec 4, 2013
Use premis hasOriginalName (and hasSize) predicates on fedora:binary resources

Resolves: https://www.pivotaltracker.com/story/show/61500736
@awoods awoods merged commit 5213396 into master Dec 4, 2013
@awoods awoods deleted the content-disposition branch December 4, 2013 07:02
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.

None yet

3 participants