|Reported by:||zooko||Owned by:|
|Cc:||zooko, zooko@…, thijs||Branch:||
(diff, github, buildbot, log)
Description (last modified by exarkun)
twisted/python/randbytes.py doesn't seem to offer a lot of functionality over os.urandom, except for two misfeatures:
- The option of setting fallback=True so that it will automatically fallback to insecure random numbers if it can't generate secure ones. This is a terrible idea that nobody should ever do. Behavior like that is one of the causes of the Debian OpenSSL Fiasco, for example. If anyone has a legitimate use case for this behavior I would be fascinated to hear it.
- The option of using PyCrypto's Crypto.Util.randpool if it is present. randpool is deprecated (according to the announcement at the top of http://www.dlitz.net/software/pycrypto/ on this date, which references this mailing list thread). I've looked at randpool a bit, and it doesn't offer any advantage over os.urandom that I can see other than the option of falling back to insecure random number generation if it can't generate secure random numbers. randpool also has a lot of other code to do some useless things about estimating entropy, sampling the current clock, and so on. I can't be sure that I understood its source code because that other stuff made it hard to understand the part I was interested in. The latest git version of PyCrypto comes with this warning: "Deprecated. Use Random.new() instead. See http://www.pycrypto.org/randpool-broken". However that link gives me a 404 Not Found.
I suspect both of these modules predate the os.urandom module that was introduced in Python 2.4. Now that Twisted no longer supports Python 2.3, perhaps the time has come to deprecate twisted.python.randbytes in favor of os.urandom.
Change History (11)
comment:1 follow-up: ↓ 3 Changed 4 years ago by exarkun
- Description modified (diff)
- Summary changed from twisted.python.randpool to twisted.python.randbytes
comment:2 Changed 4 years ago by exarkun
- Resolution set to invalid
- Status changed from new to closed
comment:3 in reply to: ↑ 1 Changed 4 years ago by thijs
- Cc thijs added
- Resolution invalid deleted
- Status changed from closed to reopened
comment:5 Changed 4 years ago by glyph
- Owner changed from glyph to zooko
- Status changed from reopened to new