amiri/Digest-PBKDF2 — pull request #3 by LoonyPandora (2012-06-07 15:24:44+00:00)
Title: Add iteration count parameter

The iteration count is analogous to the ``cost`` parameter in ``Digest::Bcrypt`` and I think it would be useful to able to set it in this module. The default imposed by ``Crypt::PBKDF2`` is only 1,000 - which is rather low in 2012 (f.ex iOS uses 10,000 iterations by default)

I also believe that it should be a required parameter, rather than defaulting it. If someone relies on the default being 1,000 and that changes upstream in the future, with no way to set it via this module, a whole bunch of user passwords could become unretrievable.

Please consider this pull request the start of a discussion, I think this parameter would be useful - but is not essential, and it does have downsides (the breaking of backwards compatibility)