Convert md5 and other checksums to lowercase when setting a location
Bug #1658882 reported by
Mike Fedosin
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Glare |
Confirmed
|
Low
|
Unassigned |
Bug Description
Now when user wants to set a location to a blob he has to specify at least md5 checksum. In common case the standard doesn't differentiate in what case this string is written. In other words strings '9d385dfdba883d
Unfortunately for Glare the case the case matters and if it tries to compare to strings from example - it fails. For this reason we need to convert these checksum string to lower case before putting them in store. 'strip' also will be good.
Instead of 'md5 = blob_meta.
"
try:
md5 = blob_meta.
except KeyError:
...
"
Changed in glare: | |
importance: | Undecided → Low |
To post a comment you must log in.
Other checksum strings (sha1, sha256) should also be converted if they exist in the request body!