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
manifest filename with newline #2
Comments
jkunze
added a commit
that referenced
this issue
Apr 14, 2014
small update about percent encoding cr, lf and crlf. fixes #2
Closed
acdha
referenced
this issue
in LibraryOfCongress/bagit-spec
Jun 6, 2017
This field is not specified but its presence in the examples can cause confusion that it's a standard field. Since any app which needs a human-formatted total file size may trivially calculate it from the Payload-Oxum (also see #2) or filesystem metadata this commit removes it to avoid confusion or the implication that tools must maintain the value.
stain
pushed a commit
to stain/bagitspec
that referenced
this issue
Apr 20, 2018
…ll manifests starting with version 1.0
stain
pushed a commit
to stain/bagitspec
that referenced
this issue
Apr 20, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Over in the bagit-python repository we had a issue opened regarding a validation error for a newly created bag. The issue was tracked down to filenames that had an embedded carriage return (0x0d) in them, which made their way into the manifest, and ultimately disrupted validation.
One approach would be to prevent the creation of bags with filenames that have embedded CR, LF or CRLF. This would involve throwing an exception or error during bag creation. Another would be to allow these filenames to exist in the manifest, but to take care to encode them in such a way that doesn't disturb the line oriented format of the manifest.
I think it's in the spirit of BagIt to do the latter, accepting that some filesystems allow CR, LF and CRLF to be present in the filename.
The text was updated successfully, but these errors were encountered: