bpo-32433: Optimized HMAC digest #5023
Conversation
Doc/library/hmac.rst
Outdated
There was a problem hiding this comment.
I realize digestmod matches the parameter name in hmac.new, but it is an antiquated name from the days when the only two hash functions were top level modules. how about just calling it digest on the new API?
Are there any hash function modules with a new method left anywhere that anyone actually uses? Why continue to support that old idiom in new APIs?
There was a problem hiding this comment.
I though about it but then decided to keep the new function compatible to the old PEP.
Since you also dislike the old name, let's use digest in the new function.
Lib/test/test_hmac.py
Outdated
There was a problem hiding this comment.
from unittest import mock
...
with mock.patch('hmac._openssl_md_meths', {}):
...
tiran
force-pushed
the
fast_hmac
branch
2 times, most recently
from
0100b1c to
7fd218c
Compare
|
@gpshead Are you ok with the new code? I'd like to get the feature into 3.7. |
gpshead
left a comment
gpshead
left a comment
There was a problem hiding this comment.
a couple comments, you can address those in this PR or in follow up minor commits as you see fit.
Lib/hmac.py
Outdated
There was a problem hiding this comment.
describe the parameters so that interactive users doing help(hmac.digest) see something useful.
There was a problem hiding this comment.
Is the "for short messages" part true? does anything care how long the message is? I'd just trim those words from the sentence. :)
There was a problem hiding this comment.
"short" for some arbitrary measurement. After all the message has to fit into memory. Anything up to 64kB should be fine. :p
The hmac module now has hmac.digest(), which provides an optimized HMAC digest for short messages. hmac.digest() is up to three times faster than hmac.HMAC().digest(). Signed-off-by: Christian Heimes <[email protected]>
|
@tiran: Please replace |
4 participants
The hmac module now has hmac.digest(), which provides an optimized HMAC
digest for short messages. hmac.digest() is up to three times faster
than hmac.HMAC().digest().
Signed-off-by: Christian Heimes [email protected]
https://bugs.python.org/issue32433