https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342021-05-03T16:39:35ZRedmine Entr’ouvertPasserelle - Bug #53697: échec de test_esirius.test_pre_request[xxx]https://dev.entrouvert.org/issues/53697?journal_id=3038082021-05-03T16:39:35ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Description</strong> mis à jour (<a title="Voir les différences" href="/journals/303808/diff?detail_id=266619">diff</a>)</li></ul> Passerelle - Bug #53697: échec de test_esirius.test_pre_request[xxx]https://dev.entrouvert.org/issues/53697?journal_id=3038122021-05-03T16:43:51ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Lié à</strong> <i><a class="issue tracker-2 status-5 priority-4 priority-default closed" href="/issues/53594">Development #53594</a>: TypeError: Object of type builtin_function_or_method is not JSON serializable</i> ajouté</li></ul> Passerelle - Bug #53697: échec de test_esirius.test_pre_request[xxx]https://dev.entrouvert.org/issues/53697?journal_id=3038402021-05-04T07:31:08ZNicolas Roche (absent jusqu'au 3 avril)
<ul><li><strong>Assigné à</strong> changé de <i>Benjamin Dauvergne</i> à <i>Nicolas Roche (absent jusqu'au 3 avril)</i></li></ul><p>Ça me semble lié à la version de cryptodome : ça passe avec <br /><pre>
$ pip install pycryptodomex==3.9.9
</pre></p>
<p>alors que sinon ça ne passe plus, même si je reviens au commit qui introduit le connecteur.</p>
<p>Dans le test je coupe la partie de la chaîne crypté liée à l'heure du chiffrement (pour avoir un test reproductible sans avoir à utiliser freeze pour autant).<br />Il semblerait que cette partie prend 10 octets de plus, j'essaye de comprendre pourquoi...</p> Passerelle - Bug #53697: échec de test_esirius.test_pre_request[xxx]https://dev.entrouvert.org/issues/53697?journal_id=3038632021-05-04T08:08:19ZNicolas Roche (absent jusqu'au 3 avril)
<ul><li><strong>Fichier</strong> <a href="/attachments/54277">0001-esirius-ignore-DES-timestamp-within-tests-53697.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/54277/0001-esirius-ignore-DES-timestamp-within-tests-53697.patch">0001-esirius-ignore-DES-timestamp-within-tests-53697.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>Nouveau</i> à <i>Solution proposée</i></li><li><strong>Patch proposed</strong> changé de <i>Non</i> à <i>Oui</i></li></ul><p>Je n'ai réussi pas trouver le commit qui explique la régression sur le test.<br />J'imagine juste quelque chose comme <a class="external" href="https://github.com/Legrandin/pycryptodome/commit/853054937e29e6cd3dc61228be1dc768b688460b#diff-fe8b2c4039525ca3fead7acd9867edecb9f87d92764718533fa68f2f2e81d1f5">https://github.com/Legrandin/pycryptodome/commit/853054937e29e6cd3dc61228be1dc768b688460b#diff-fe8b2c4039525ca3fead7acd9867edecb9f87d92764718533fa68f2f2e81d1f5</a></p>
<p>Dans les tests il y en a un autre qui valide le token complet (test_token : avec une date fixée par freeze) et donc cette erreur ne montre pas une régression.<br />Le test qui plante ici ne teste que la partie supposée fixe du token.<br />Il est redondant et on pourrait même le retirer.</p> Passerelle - Bug #53697: échec de test_esirius.test_pre_request[xxx]https://dev.entrouvert.org/issues/53697?journal_id=3039782021-05-04T11:04:42ZBenjamin Dauvergne
<ul><li><strong>Planning</strong> changé de <i>Non</i> à <i>Oui</i></li></ul><p>Tu ne veux pas juste décoder le token et vérifier ce qui doit l'être ? Ça me semble plus sûr.</p> Passerelle - Bug #53697: échec de test_esirius.test_pre_request[xxx]https://dev.entrouvert.org/issues/53697?journal_id=3040302021-05-04T12:50:52ZNicolas Roche (absent jusqu'au 3 avril)
<ul><li><strong>Fichier</strong> <a href="/attachments/54312">0001-esirius-ignore-DES-timestamp-within-tests-53697.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/54312/0001-esirius-ignore-DES-timestamp-within-tests-53697.patch">0001-esirius-ignore-DES-timestamp-within-tests-53697.patch</a> ajouté</li></ul><p>(fait)</p> Passerelle - Bug #53697: échec de test_esirius.test_pre_request[xxx]https://dev.entrouvert.org/issues/53697?journal_id=3041202021-05-04T14:29:10ZBenjamin Dauvergne
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul> Passerelle - Bug #53697: échec de test_esirius.test_pre_request[xxx]https://dev.entrouvert.org/issues/53697?journal_id=3041562021-05-04T15:17:07ZNicolas Roche (absent jusqu'au 3 avril)
<ul><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>Résolu (à déployer)</i></li></ul><p>(quelqu'un l'a déjà poussé pour moi)<br /><pre>
commit 59186c7cf9bb9c6adaeca379dea8947f75ee98ee
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date: Tue May 4 09:56:32 2021 +0200
esirius: ignore DES timestamp within tests (#53697)
</pre></p> Passerelle - Bug #53697: échec de test_esirius.test_pre_request[xxx]https://dev.entrouvert.org/issues/53697?journal_id=3042602021-05-04T21:17:02ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>Résolu (à déployer)</i> à <i>Solution déployée</i></li></ul>