Code review comment for lp:~jaypipes/glance/checksum
Revision history for this message
| Jay Pipes (jaypipes) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
73416ba
(Get the code!)
> > 110 + checksum = Column(String(32))
>
> I assume the answer is YAGNI, but I'll ask it anyway: Will we ever allow SHA1
> or SHA256?
>
> Would it make sense to make this a String(64) or, heck, a String(255) for
> flexibility?
Hmm, good question. Not sure, really. I suppose we could expand it in the future if we decide its a needed feature.