Code review comment for lp:~jaypipes/glance/checksum

Revision history for this message
Chuck Thier (cthier) wrote :

The swift parts look reasonable to me. The only other thing that I might add is to clarify in the docs that the checksum is an md5 checksum, otherwise people may try to use CRC.

review: Approve

« Back to merge proposal