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

Checksum comparison should be case insensitive #142

Open
luis100 opened this issue Mar 29, 2023 · 0 comments
Open

Checksum comparison should be case insensitive #142

luis100 opened this issue Mar 29, 2023 · 0 comments

Comments

@luis100
Copy link

luis100 commented Mar 29, 2023

Version: 5.2.0
OS: Ubuntu 22.04.2 LTS

Bagit fails to load due to validation issue:

gov.loc.repository.bagit.exceptions.CorruptChecksumException: File [-redacted-.pdf] is suppose to have a [MD5] hash of [8F7EF44A45A91BF762D122E6C8A2C4A5] but was computed [8f7ef44a45a91bf762d122e6c8a2c4a5].

But the RFC states.

o The hex-encoded checksum MAY use uppercase and/or lowercase letters.

So comparison should be case insensitive.
luis100 added a commit to luis100/bagit-java that referenced this issue Mar 29, 2023
@luis100
Make checksum comparison case insensitive, fixes LibraryOfCongress#142
d1731e1
jscancella added a commit to jscancella/bagging that referenced this issue Mar 29, 2023
@jscancella
fix: don't compare case of hash
e14b203 
see LibraryOfCongress/bagit-java#142
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@luis100