https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342019-08-05T16:41:51ZRedmine Entr’ouvertCombo - Development #35225: requests_wrapper: intégrer params à l'URL avant la signaturehttps://dev.entrouvert.org/issues/35225?journal_id=1860462019-08-05T16:41:51ZBenjamin Dauvergne
<ul><li><strong>Sujet</strong> changé de <i>newsletters: passer les paramètres uuid et mobile dans l'url avant la signature</i> à <i>requests_wrapper: intégrer params à l'URL avant la signature</i></li></ul><p>En fait c'est combo.utils.requests_wrapper.Requests qui doit être corrigé.</p> Combo - Development #35225: requests_wrapper: intégrer params à l'URL avant la signaturehttps://dev.entrouvert.org/issues/35225?journal_id=1860472019-08-05T16:45:26ZSerghei Mihai
<ul></ul><p>C'est ce que je me suis dit en regardant notre code qui surcharge <code>requests</code>.</p> Combo - Development #35225: requests_wrapper: intégrer params à l'URL avant la signaturehttps://dev.entrouvert.org/issues/35225?journal_id=1860482019-08-05T17:09:54ZBenjamin Dauvergne
<ul><li><strong>Assigné à</strong> changé de <i>Serghei Mihai</i> à <i>Benjamin Dauvergne</i></li></ul> Combo - Development #35225: requests_wrapper: intégrer params à l'URL avant la signaturehttps://dev.entrouvert.org/issues/35225?journal_id=1860492019-08-05T17:09:54ZBenjamin Dauvergne
<ul><li><strong>Fichier</strong> <a href="/attachments/36426">0001-requests_wrapper-sign-URL-of-prepared-requests-35225.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/36426/0001-requests_wrapper-sign-URL-of-prepared-requests-35225.patch">0001-requests_wrapper-sign-URL-of-prepared-requests-35225.patch</a> ajouté</li><li><strong>Tracker</strong> changé de <i>Bug</i> à <i>Development</i></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> Combo - Development #35225: requests_wrapper: intégrer params à l'URL avant la signaturehttps://dev.entrouvert.org/issues/35225?journal_id=1860802019-08-06T08:12:27ZSerghei Mihai
<ul><li><strong>Fichier</strong> <a href="/attachments/36440">0001-requests_wrapper-sign-URL-of-prepared-requests-35225.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/36440/0001-requests_wrapper-sign-URL-of-prepared-requests-35225.patch">0001-requests_wrapper-sign-URL-of-prepared-requests-35225.patch</a> ajouté</li></ul><p>Benj, j'ai adapté les autres tests et rajouté un check que la signature est bien le dernier paramètre dans l'url.<br />Je te laisse dire quoi.</p> Combo - Development #35225: requests_wrapper: intégrer params à l'URL avant la signaturehttps://dev.entrouvert.org/issues/35225?journal_id=1860892019-08-06T09:12:36ZSerghei Mihai
<ul><li><strong>Fichier</strong> <a href="/attachments/36444">0001-requests_wrapper-sign-URL-of-prepared-requests-35225.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/36444/0001-requests_wrapper-sign-URL-of-prepared-requests-35225.patch">0001-requests_wrapper-sign-URL-of-prepared-requests-35225.patch</a> ajouté</li></ul><p>J'ai poussé la branche à jour avec le patch ammendé.</p> Combo - Development #35225: requests_wrapper: intégrer params à l'URL avant la signaturehttps://dev.entrouvert.org/issues/35225?journal_id=1860922019-08-06T09:17:59ZBenjamin Dauvergne
<ul><li><strong>Fichier</strong> <a href="/attachments/36445">0001-requests_wrapper-sign-URL-of-prepared-requests-35225.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/36445/0001-requests_wrapper-sign-URL-of-prepared-requests-35225.patch">0001-requests_wrapper-sign-URL-of-prepared-requests-35225.patch</a> ajouté</li></ul><p>Patch avec correction aux autres tests.</p> Combo - Development #35225: requests_wrapper: intégrer params à l'URL avant la signaturehttps://dev.entrouvert.org/issues/35225?journal_id=1860992019-08-06T09:28:06ZThomas Noël
<ul></ul><p>Pour vérifier la correction faite par le patch, il faudrait vérifier que la requête produite par test_get_subscriptions_signature_check est correctement signée (ce qui ne doit pas être le cas sans le patch, à cause des arguments ajoutés après signature)</p> Combo - Development #35225: requests_wrapper: intégrer params à l'URL avant la signaturehttps://dev.entrouvert.org/issues/35225?journal_id=1861072019-08-06T09:31:16ZBenjamin Dauvergne
<ul></ul><p>Thomas Noël a écrit :</p>
<blockquote>
<p>Pour vérifier la correction faite par le patch, il faudrait vérifier que la requête produite par test_get_subscriptions_signature_check est correctement signée (ce qui ne doit pas être le cas sans le patch, à cause des arguments ajoutés après signature)</p>
</blockquote>
<p>C'est vérifié dans mon patch.</p> Combo - Development #35225: requests_wrapper: intégrer params à l'URL avant la signaturehttps://dev.entrouvert.org/issues/35225?journal_id=1861102019-08-06T09:35:03ZBenjamin Dauvergne
<ul><li><strong>Fichier</strong> <a href="/attachments/36447">0001-requests_wrapper-sign-URL-of-prepared-requests-35225.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/36447/0001-requests_wrapper-sign-URL-of-prepared-requests-35225.patch">0001-requests_wrapper-sign-URL-of-prepared-requests-35225.patch</a> ajouté</li></ul><p>Voilà, avec un check explicite.</p> Combo - Development #35225: requests_wrapper: intégrer params à l'URL avant la signaturehttps://dev.entrouvert.org/issues/35225?journal_id=1861202019-08-06T09:56:34ZSerghei Mihai
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul><p>Go.</p> Combo - Development #35225: requests_wrapper: intégrer params à l'URL avant la signaturehttps://dev.entrouvert.org/issues/35225?journal_id=1861232019-08-06T10:04:32ZBenjamin Dauvergne
<ul><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>Résolu (à déployer)</i></li></ul><pre>commit 546336acc75f1922929b01b88463d54d568a5eea
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Mon Aug 5 18:57:06 2019 +0200
requests_wrapper: sign URL of prepared requests (#35225)</pre> Combo - Development #35225: requests_wrapper: intégrer params à l'URL avant la signaturehttps://dev.entrouvert.org/issues/35225?journal_id=1861512019-08-06T12:44:55ZSerghei Mihai
<ul><li><strong>Statut</strong> changé de <i>Résolu (à déployer)</i> à <i>Solution déployée</i></li></ul> Combo - Development #35225: requests_wrapper: intégrer params à l'URL avant la signaturehttps://dev.entrouvert.org/issues/35225?journal_id=1861642019-08-06T13:18:22ZSerghei Mihai
<ul></ul><p>Commit poussé dans la branch <code>hotfix/v2.26</code>, à envoyer en prod ce soir.</p>