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.

Sign up for GitHub

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

Jump to bottom

We should say that manifest digests MUST match the files #455

Closed
zimeon opened this issue May 5, 2020 · 1 comment · Fixed by #458
Closed

We should say that manifest digests MUST match the files #455

zimeon opened this issue May 5, 2020 · 1 comment · Fixed by #458
Assignees
@zimeon
Labels
Editorial Editorial issues (no changes to intent) OCFL Object
Milestone
1.0

Comments

@zimeon
Copy link
Contributor

zimeon commented May 5, 2020

By my reading of the description of the manifest and elsewhere we don't actually say that the manifest digest values MUST match the digests for the corresponding files listed.

(I'm pleased to say we do for Inventory Digest https://ocfl.io/draft/spec/#E060 . However, we don't for Fixity https://ocfl.io/draft/spec/#fixity --> need separate issue)
@zimeon zimeon added Editorial Editorial issues (no changes to intent) OCFL Object Needs Discussion labels May 5, 2020
@zimeon zimeon mentioned this issue May 5, 2020
Closed
@rosy1280 rosy1280 added this to the 1.0 milestone May 5, 2020
@zimeon
Copy link
Contributor Author

zimeon commented May 5, 2020

Editors' meeting - agree should fix

@zimeon zimeon mentioned this issue May 5, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zimeon zimeon

Labels
Editorial Editorial issues (no changes to intent) OCFL Object
Projects
None yet
Milestone
1.0
Development

Successfully merging a pull request may close this issue.

Manifest digest values MUST match the content digests OCFL/spec
2 participants
@rosy1280 @zimeon