https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342020-06-22T08:09:00ZRedmine Entr’ouvertAuthentic 2 - Bug #44301: Prendre en compte l'exception MultipleObjectsReturned dans l'implémentation get/update-or-create de l'API (était MultipleObjectsReturned: get() returned more than one User -- it returned 2!)https://dev.entrouvert.org/issues/44301?journal_id=2436522020-06-22T08:09:00ZLauréline Guérin
<ul><li><strong>Projet</strong> changé de <i>Suivi des traces</i> à <i>Authentic 2</i></li></ul> Authentic 2 - Bug #44301: Prendre en compte l'exception MultipleObjectsReturned dans l'implémentation get/update-or-create de l'API (était MultipleObjectsReturned: get() returned more than one User -- it returned 2!)https://dev.entrouvert.org/issues/44301?journal_id=2436762020-06-22T09:02:35ZBenjamin Dauvergne
<ul><li><strong>Sujet</strong> changé de <i>MultipleObjectsReturned: get() returned more than one User -- it returned 2!</i> à <i>Prendre en compte l'exception MultipleObjectsReturned dans l'implémentation get/update-or-create de l'API (était MultipleObjectsReturned: get() returned more than one User -- it returned 2!)</i></li></ul> Authentic 2 - Bug #44301: Prendre en compte l'exception MultipleObjectsReturned dans l'implémentation get/update-or-create de l'API (était MultipleObjectsReturned: get() returned more than one User -- it returned 2!)https://dev.entrouvert.org/issues/44301?journal_id=2436772020-06-22T09:03:40ZBenjamin Dauvergne
<ul></ul><p>On ne récupère pas MultipleObjectsReturned dans authentic2.api_mixins.GetOrCreateMixinView._lookup_instance, il faudrait.</p> Authentic 2 - Bug #44301: Prendre en compte l'exception MultipleObjectsReturned dans l'implémentation get/update-or-create de l'API (était MultipleObjectsReturned: get() returned more than one User -- it returned 2!)https://dev.entrouvert.org/issues/44301?journal_id=2512162020-07-30T10:27:18ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Dupliqué par</strong> <i><a class="issue tracker-1 status-6 priority-4 priority-default closed" href="/issues/45594">Bug #45594</a>: MultipleObjectsReturned: get() returned more than one User -- it returned 2!</i> ajouté</li></ul> Authentic 2 - Bug #44301: Prendre en compte l'exception MultipleObjectsReturned dans l'implémentation get/update-or-create de l'API (était MultipleObjectsReturned: get() returned more than one User -- it returned 2!)https://dev.entrouvert.org/issues/44301?journal_id=2512202020-07-30T10:31:39ZPierre Crospcros@entrouvert.com
<ul></ul><p>C'est critique au Nord, merci.</p> Authentic 2 - Bug #44301: Prendre en compte l'exception MultipleObjectsReturned dans l'implémentation get/update-or-create de l'API (était MultipleObjectsReturned: get() returned more than one User -- it returned 2!)https://dev.entrouvert.org/issues/44301?journal_id=2512272020-07-30T11:05:50ZPaul Marillonnet
<ul><li><strong>Lié à</strong> <i><a class="issue tracker-1 status-5 priority-4 priority-default closed" href="/issues/45199">Bug #45199</a>: auth_fc: création d'un doublon même si un compte avec l'adresse mail envoyée par FC existe</i> ajouté</li></ul> Authentic 2 - Bug #44301: Prendre en compte l'exception MultipleObjectsReturned dans l'implémentation get/update-or-create de l'API (était MultipleObjectsReturned: get() returned more than one User -- it returned 2!)https://dev.entrouvert.org/issues/44301?journal_id=2513222020-07-30T15:39:43ZPaul Marillonnet
<ul><li><strong>Fichier</strong> <a href="/attachments/46766">0001-api-handle-multiple-objects-returned-in-get-or-creat.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/46766/0001-api-handle-multiple-objects-returned-in-get-or-creat.patch">0001-api-handle-multiple-objects-returned-in-get-or-creat.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> Authentic 2 - Bug #44301: Prendre en compte l'exception MultipleObjectsReturned dans l'implémentation get/update-or-create de l'API (était MultipleObjectsReturned: get() returned more than one User -- it returned 2!)https://dev.entrouvert.org/issues/44301?journal_id=2513622020-07-30T22:38:31ZThomas Noël
<ul></ul><p>Ça m'a l'air bien, en tout cas le test est très clair, même si 400 (Bad Request) n'est pas le statut que j'attendais. Mais on peut imaginer que ça veuille dire "attention tu demandes un email unique mais en fait c'est juste de la chance si ça marche car il n'y a pas de contrainte à ce niveau".</p>
<p>Et dans ce cas, on se dit qu'on ne devrait jamais autoriser un get_or_create=email quand l'email n'est pas configuré pour être unique, et toujours faire un 400 dans ce cas...?</p>
<p>(J'aime bien remuer la vase)</p> Authentic 2 - Bug #44301: Prendre en compte l'exception MultipleObjectsReturned dans l'implémentation get/update-or-create de l'API (était MultipleObjectsReturned: get() returned more than one User -- it returned 2!)https://dev.entrouvert.org/issues/44301?journal_id=2515152020-07-31T14:29:59ZPaul Marillonnet
<ul><li><strong>Fichier</strong> <a href="/attachments/46826">0001-api-handle-multiple-objects-returned-in-get-or-creat.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/46826/0001-api-handle-multiple-objects-returned-in-get-or-creat.patch">0001-api-handle-multiple-objects-returned-in-get-or-creat.patch</a> ajouté</li></ul><p>Thomas Noël a écrit :</p>
<blockquote>
<p>Ça m'a l'air bien, en tout cas le test est très clair, même si 400 (Bad Request) n'est pas le statut que j'attendais. Mais on peut imaginer que ça veuille dire "attention tu demandes un email unique mais en fait c'est juste de la chance si ça marche car il n'y a pas de contrainte à ce niveau".</p>
</blockquote>
<p>Nouveau patch avec un code d'erreur 409 (conflit sur la ressource demandée).</p>
<blockquote>
<p>Et dans ce cas, on se dit qu'on ne devrait jamais autoriser un get_or_create=email quand l'email n'est pas configuré pour être unique, et toujours faire un 400 dans ce cas...?</p>
</blockquote>
<p>Je ne sais pas si l'API avait été conçue pour que les attributs de get_or_create forment une clé naturelle. Est-ce aberrant de demander à l'API "trouve moi un Jean-Michel Dupont s'il n'y en a qu'un seul, et s'il n'existe pas, tiens justement en voilà un." ?</p> Authentic 2 - Bug #44301: Prendre en compte l'exception MultipleObjectsReturned dans l'implémentation get/update-or-create de l'API (était MultipleObjectsReturned: get() returned more than one User -- it returned 2!)https://dev.entrouvert.org/issues/44301?journal_id=2515242020-07-31T14:57:00ZThomas Noël
<ul><li><strong>Planning</strong> changé de <i>Non</i> à <i>Oui</i></li></ul><p>J'ai l'impression que ça fait bien le job. Mais je suis perdu dans les questions métaphysiques d'usage de ce get_or_create, alors je préfère laisser un autre oeil relire aussi.</p>
Détails :
<ul>
<li>habituellement on ne met pas à jour le copyright, je ne sais pas si faut ou pas</li>
<li>j'aurais posé Conflit dans api_mixins parce qu'ajouter des fichiers qui déclarent juste un petit truc utilisé une seule fois, bof</li>
</ul> Authentic 2 - Bug #44301: Prendre en compte l'exception MultipleObjectsReturned dans l'implémentation get/update-or-create de l'API (était MultipleObjectsReturned: get() returned more than one User -- it returned 2!)https://dev.entrouvert.org/issues/44301?journal_id=2515372020-07-31T15:24:41ZPaul Marillonnet
<ul><li><strong>Fichier</strong> <a href="/attachments/46835">0001-api-handle-multiple-objects-returned-in-get-or-creat.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/46835/0001-api-handle-multiple-objects-returned-in-get-or-creat.patch">0001-api-handle-multiple-objects-returned-in-get-or-creat.patch</a> ajouté</li></ul><p>Thomas Noël a écrit :</p>
<blockquote>
<p>J'ai l'impression que ça fait bien le job. Mais je suis perdu dans les questions métaphysiques d'usage de ce get_or_create, alors je préfère laisser un autre oeil relire aussi.</p>
</blockquote>
<p>(Oui en remuant la vase on finit par s'y perdre ☺)</p>
<blockquote>
Détails :
<ul>
<li>habituellement on ne met pas à jour le copyright, je ne sais pas si faut ou pas</li>
</ul>
</blockquote>
<p>Ok, dans le doute je ne mets pas à jour.</p>
<blockquote>
<ul>
<li>j'aurais posé Conflit dans api_mixins parce qu'ajouter des fichiers qui déclarent juste un petit truc utilisé une seule fois, bof</li>
</ul>
</blockquote>
<p>Ok.</p> Authentic 2 - Bug #44301: Prendre en compte l'exception MultipleObjectsReturned dans l'implémentation get/update-or-create de l'API (était MultipleObjectsReturned: get() returned more than one User -- it returned 2!)https://dev.entrouvert.org/issues/44301?journal_id=2590412020-09-18T12:42:38ZBenjamin Dauvergne
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul> Authentic 2 - Bug #44301: Prendre en compte l'exception MultipleObjectsReturned dans l'implémentation get/update-or-create de l'API (était MultipleObjectsReturned: get() returned more than one User -- it returned 2!)https://dev.entrouvert.org/issues/44301?journal_id=2590562020-09-18T13:07:36ZBenjamin Dauvergne
<ul><li><strong>Assigné à</strong> mis à <i>Benjamin Dauvergne</i></li></ul> Authentic 2 - Bug #44301: Prendre en compte l'exception MultipleObjectsReturned dans l'implémentation get/update-or-create de l'API (était MultipleObjectsReturned: get() returned more than one User -- it returned 2!)https://dev.entrouvert.org/issues/44301?journal_id=2590572020-09-18T13:07:36ZBenjamin Dauvergne
<ul><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>Résolu (à déployer)</i></li></ul><pre>commit 52c21f7f3e15a32b3f2ec12e56155197baac4cbd
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date: Thu Jul 30 17:31:00 2020 +0200
api: handle multiple objects returned in get-or-create mixin (#44301)</pre> Authentic 2 - Bug #44301: Prendre en compte l'exception MultipleObjectsReturned dans l'implémentation get/update-or-create de l'API (était MultipleObjectsReturned: get() returned more than one User -- it returned 2!)https://dev.entrouvert.org/issues/44301?journal_id=2595082020-09-21T16:15:52ZFré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>