Code review comment for lp:~tjoneslo/akiban-server/add-rest-model-hash

Revision history for this message
Nathan Williams (nwilliams) wrote :

I think formatMD5 needs another parameter for upper vs lowercase to preserve the SercurityService needs. Just moving the old SecurityService implementation to Strings would probably suffice.

review: Needs Fixing

« Back to merge proposal