Development #43563
Des tests de combo sont cassés en python3.8
0%
Description
$ tox -re py3-django22 -- tests/test_utils.py ... > t = time.clock() E AttributeError: module 'time' has no attribute 'clock' /tmp/tox-nroche/combo/py3-django22/lib/python3.8/site-packages/Crypto/Random/_UserFriendlyRNG.py:77: AttributeError
cf https://docs.python.org/3/whatsnew/3.8.html#api-and-feature-removals :
The function time.clock() has been removed, after having been deprecated since Python 3.3: use time.perf_counter() or time.process_time() instead, depending on your requirements, to have well-defined behavior. (Contributed by Matthias Bussonnier in bpo-36895.)
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Nicolas Roche il y a presque 4 ans
La librairie pycrypto ne propose pas de nouvelle version (depuis 2013) :
https://pypi.org/project/pycrypto/#history
https://github.com/pycrypto/pycrypto/blob/master/ChangeLog
- passerelle/apps/cryptor/models.py
- authentic/src/authentic2/crypto.py avec du code très similaire ici.
Mis à jour par Frédéric Péters il y a presque 4 ans
- Sujet changé de Les tests de combo sont cassés en python3.8 à Des tests de combo sont cassés en python3.8
Mis à jour par Nicolas Roche il y a presque 4 ans
- Lié à Bug #38017: authentic2_auth_fc:Plugin ImportError('No module named Crypto.Cipher',) ajouté
Mis à jour par Nicolas Roche il y a presque 4 ans
- Fichier 0001-utils-use-pycrytodomex-replace-Crypto-with-Cryptodom.patch 0001-utils-use-pycrytodomex-replace-Crypto-with-Cryptodom.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Note: J'ai du modifier le code pour ne pas introduire de régression dans les tests
(alors que cette modification n'est pas faite sur authentic).
def aes_hex_encrypt(key, data):
iv = Random.get_random_bytes(2) * 8
aes_key = PBKDF2(key, iv)
aes = AES.new(aes_key, AES.MODE_CFB, iv)
- crypted = aes.encrypt(data)
+ crypted = aes.encrypt(force_bytes(data))
return force_text(b'%s%s' % (binascii.hexlify(iv[:2]), binascii.hexlify(crypted)))
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
Nicolas Roche a écrit :
Note: J'ai du modifier le code pour ne pas introduire de régression dans les tests
(alors que cette modification n'est pas faite sur authentic).
Coté authentic l'API suppose qu'on ne lui passe que des bytes (je trouve ça mieux ainsi personnellement, chiffrer c'est pas niveau) :
data['block']['encrypted_bindpw'] = force_text(crypto.aes_base64_encrypt(
settings.SECRET_KEY, force_bytes(data['block']['bindpw'])))
Mis à jour par Nicolas Roche il y a presque 4 ans
- Fichier 0001-utils-use-pycrytodomex-replace-Crypto-with-Cryptodom.patch 0001-utils-use-pycrytodomex-replace-Crypto-with-Cryptodom.patch ajouté
Voici une nouvelle version faisant en sorte qu'aes_hex_encrypt soit toujours appelée en lui passant des bytes.
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Solution proposée à Résolu (à déployer)
Validé/poussé, corrigé cryto -> crypto dans le message.
commit 7494896e9faa8501b24ffe3a32dc9a0d7df17a73
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date: Wed Jun 3 11:27:09 2020 +0200
utils: switch to pycryptodomex, replace Crypto with Cryptodome (#43563)
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
utils: switch to pycryptodomex, replace Crypto with Cryptodome (#43563)