buzzfeed / mattress

iOS Offline Caching for Web Content
MIT License
520 stars 45 forks source link

Use CommonCrypto instead of CryptoSwift #5

Closed lordkev closed 9 years ago

lordkev commented 9 years ago

Based this on the following SO answers. It uses a dummy framework to import CommonCrypto so that the end user of Mattress doesn't have to add the module map to their own project.

http://stackoverflow.com/questions/24123518/how-to-use-cc-md5-method-in-swift-language

http://stackoverflow.com/questions/25248598/importing-commoncrypto-in-a-swift-framework

sharpfive commented 9 years ago

LGTM. Nice work.

dmauro commented 9 years ago

This is awesome, thanks! Validates in the News app, so I'll merge this in here and then we should see about getting this branch into master.