https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342019-04-12T07:41:14ZRedmine Entr’ouvertEOPayment - Development #32224: créer une nouvelle API guess() sur les backends pour permettre de détecter un backend depuis une réponsehttps://dev.entrouvert.org/issues/32224?journal_id=1671772019-04-12T07:41:14ZBenjamin Dauvergne
<ul><li><strong>Lié à</strong> <i><a class="issue tracker-2 status-5 priority-4 priority-default closed" href="/issues/32194">Development #32194</a>: Paiement : transaction à validation manuelle sur payzen</i> ajouté</li></ul> EOPayment - Development #32224: créer une nouvelle API guess() sur les backends pour permettre de détecter un backend depuis une réponsehttps://dev.entrouvert.org/issues/32224?journal_id=1681362019-04-16T07:53:58ZEmmanuel Cazenaveecazenave@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>Nouveau</i> à <i>En cours</i></li><li><strong>Assigné à</strong> mis à <i>Emmanuel Cazenave</i></li></ul> EOPayment - Development #32224: créer une nouvelle API guess() sur les backends pour permettre de détecter un backend depuis une réponsehttps://dev.entrouvert.org/issues/32224?journal_id=3892712022-06-24T11:06:13ZBenjamin Dauvergne
<ul><li><strong>Assigné à</strong> changé de <i>Emmanuel Cazenave</i> à <i>Benjamin Dauvergne</i></li></ul> EOPayment - Development #32224: créer une nouvelle API guess() sur les backends pour permettre de détecter un backend depuis une réponsehttps://dev.entrouvert.org/issues/32224?journal_id=3892722022-06-24T11:06:13ZBenjamin Dauvergne
<ul><li><strong>Fichier</strong> <a href="/attachments/66974">0003-add-a-method-to-guess-transaction_id-and-backend-fro.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/66974/0003-add-a-method-to-guess-transaction_id-and-backend-fro.patch">0003-add-a-method-to-guess-transaction_id-and-backend-fro.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/66975">0001-misc-remove-references-to-sips-32224.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/66975/0001-misc-remove-references-to-sips-32224.patch">0001-misc-remove-references-to-sips-32224.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/66976">0002-misc-remove-dead-code-32224.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/66976/0002-misc-remove-dead-code-32224.patch">0002-misc-remove-dead-code-32224.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>En cours</i> à <i>Solution proposée</i></li><li><strong>Patch proposed</strong> changé de <i>Non</i> à <i>Oui</i></li></ul> EOPayment - Development #32224: créer une nouvelle API guess() sur les backends pour permettre de détecter un backend depuis une réponsehttps://dev.entrouvert.org/issues/32224?journal_id=3901922022-06-28T12:13:05ZEmmanuel Cazenaveecazenave@entrouvert.com
<ul><li><strong>Planning</strong> mis à <i>Non</i></li></ul><p>Pour faciliter la compréhension, c'est quoi le cas d'usage ?</p>
<p>La description mentionne <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: Paiement : transaction à validation manuelle sur payzen (Fermé)" href="https://dev.entrouvert.org/issues/32194">#32194</a> qui fait 4 km de long et a débouché sur autre chose que qui est proposé ici, donc ça n'aide pas à comprendre.</p> EOPayment - Development #32224: créer une nouvelle API guess() sur les backends pour permettre de détecter un backend depuis une réponsehttps://dev.entrouvert.org/issues/32224?journal_id=3902262022-06-28T12:55:38ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><p>Via <a class="issue tracker-2 status-5 priority-3 priority-lowest closed" title="Development: obtenir un transaction_id à partir d'une réponse d'un système de paiement, sans connaitre celui-c... (Fermé)" href="https://dev.entrouvert.org/issues/56697">#56697</a> qui a été fermé pour se concentrer ici, dans celui-là je décrivais la situation :</p>
<blockquote>
<p>Ça vient de moments de grandes confusions de transfert prod/régie avec des URL à mentionner et des mélanges d'id de régie et de systèmes de paiement; l'idée derrière est de s'éviter ça en permettant de retrouver le nécessaire sur la seule base de l'id de transaction. (ce ticket fait partie d'une série, dont <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: remonter en erreur un callback vers une régie qui n'existe pas (Fermé)" href="https://dev.entrouvert.org/issues/56647">#56647</a>, créés après l'incident, pour améliorer les choses).</p>
</blockquote> EOPayment - Development #32224: créer une nouvelle API guess() sur les backends pour permettre de détecter un backend depuis une réponsehttps://dev.entrouvert.org/issues/32224?journal_id=3902702022-06-28T14:04:25ZEmmanuel Cazenaveecazenave@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul><pre>
transaction_id = backend.guess(
method=method, query_string=query_string, body=body, headers=headers
)
</pre>
<p>Je mettrais ça dans un try/except.</p> EOPayment - Development #32224: créer une nouvelle API guess() sur les backends pour permettre de détecter un backend depuis une réponsehttps://dev.entrouvert.org/issues/32224?journal_id=3902992022-06-28T15:06:45ZBenjamin Dauvergne
<ul><li><strong>Fichier</strong> <a href="/attachments/67082">0003-add-a-method-to-guess-transaction_id-and-backend-fro.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/67082/0003-add-a-method-to-guess-transaction_id-and-backend-fro.patch">0003-add-a-method-to-guess-transaction_id-and-backend-fro.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/67083">0001-misc-remove-references-to-sips-32224.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/67083/0001-misc-remove-references-to-sips-32224.patch">0001-misc-remove-references-to-sips-32224.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/67084">0004-wip.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/67084/0004-wip.patch">0004-wip.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/67085">0002-misc-remove-dead-code-32224.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/67085/0002-misc-remove-dead-code-32224.patch">0002-misc-remove-dead-code-32224.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>Solution proposée</i></li></ul><p>Patch "wip" pour implémenter l'idée de manu, ce serait effectivement dommage de casse un backend depuis un autre (mais le paramètre backends est sensé être là pour ça).</p>
<p>Donc ici, j'intercepte les exceptions, et si je ne trouve pas de transaction_id, je relance la dernière sinon BackendNotFound.</p> EOPayment - Development #32224: créer une nouvelle API guess() sur les backends pour permettre de détecter un backend depuis une réponsehttps://dev.entrouvert.org/issues/32224?journal_id=3903082022-06-28T15:27:01ZEmmanuel Cazenaveecazenave@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul><p>J'étais sur l'idée toute simple qu'un appel à whateverbackend.guess pourrait planter en cas de réponse farfelue, et de ne pas tout planter là dessus.</p>
<p>Mais relire les divers <code>guess</code>, ils me paraissent assez défensifs comme ça, laisse tomber ma remarque.</p> EOPayment - Development #32224: créer une nouvelle API guess() sur les backends pour permettre de détecter un backend depuis une réponsehttps://dev.entrouvert.org/issues/32224?journal_id=3903172022-06-28T15:55:34ZBenjamin Dauvergne
<ul><li><strong>Bloqué par</strong> <i><a class="issue tracker-2 status-5 priority-4 priority-default closed" href="/issues/66737">Development #66737</a>: lingo: retirer la référence à SIPS</i> ajouté</li></ul> EOPayment - Development #32224: créer une nouvelle API guess() sur les backends pour permettre de détecter un backend depuis une réponsehttps://dev.entrouvert.org/issues/32224?journal_id=3908072022-06-30T10:11:38ZBenjamin Dauvergne
<ul><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>Résolu (à déployer)</i></li></ul><pre>commit 7b73275f7090d8c9526f9548434df94c45416b77
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Fri Jun 24 13:04:41 2022 +0200
add a method to guess transaction_id and backend from an HTTP response (#32224)
commit fe27aeb4e5065f7cc98aeb4244ee3a4eca88c383
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Fri Jun 24 13:04:39 2022 +0200
misc: remove dead code (#32224)
commit 02d1238761602d9052d187e35c3cb278e24aaee9
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Fri Jun 24 11:38:00 2022 +0200
misc: remove references to sips (#32224)
It broke get_backends().</pre> EOPayment - Development #32224: créer une nouvelle API guess() sur les backends pour permettre de détecter un backend depuis une réponsehttps://dev.entrouvert.org/issues/32224?journal_id=3912282022-07-01T11:15:03ZTransition automatique
<ul><li><strong>Statut</strong> changé de <i>Résolu (à déployer)</i> à <i>Solution déployée</i></li></ul> EOPayment - Development #32224: créer une nouvelle API guess() sur les backends pour permettre de détecter un backend depuis une réponsehttps://dev.entrouvert.org/issues/32224?journal_id=4039202022-09-04T02:42:02ZTransition automatique
<ul></ul><p>Automatic expiration</p>