Projet

Général

Profil

Development #17171

json data store : autoriser name_id dans les actions de modification

Ajouté par Frédéric Péters il y a presque 7 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
24 juin 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

De manière optionnelle mais l'idée c'est qu'en passant name_id on assure un contrôle sur le "propriétaire" de la donnée.


Fichiers


Demandes liées

Lié à Combo - Bug #17173: assurer que user_nameid et user_email ne puissent être forgés dans les urlsFermé24 juin 2017

Actions

Révisions associées

Révision 0a48496c (diff)
Ajouté par Frédéric Péters il y a plus de 6 ans

json data store: allow name_id in edit/delete methods (#17171)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 7 ans

#2

Mis à jour par Frédéric Péters il y a presque 7 ans

  • Lié à Bug #17173: assurer que user_nameid et user_email ne puissent être forgés dans les urls ajouté
#3

Mis à jour par Thomas Noël il y a presque 7 ans

Il faut documenter la possibilité dans le details.html

(et j'insiste, avec un json-api on aurait un err:1 cadeau)

#4

Mis à jour par Frédéric Péters il y a presque 7 ans

(et j'insiste, avec un json-api on aurait un err:1 cadeau)

J'utiliserai json-api quand j'aurai eu l'occasion d'en faire le tour et de documenter les comportements automatiques. (invitation à chacun)

#5

Mis à jour par Frédéric Péters il y a plus de 6 ans

Patch revu suite à l'adoption de #17175, avec notamment la levée de DoesNotExist plutôt que forcer des 404, pour être conforme à une décision à venir.

#6

Mis à jour par Thomas Noël il y a plus de 6 ans

Dans les tests de delete, et pour préparer/montrer l'avenir, je verrais bien des « assert resp.json['err'] != 0 » pour les échecs et « assert resp.json['err'] == 0 » pour le succès.

Mais même sans ça, c'est un ack.

#7

Mis à jour par Frédéric Péters il y a plus de 6 ans

  • Statut changé de En cours à Résolu (à déployer)

Il y en avait déjà un pour un test de réussite, j'ai ajouté aux autres.

commit 0a48496c4cdc5d7c9a7ba7eed9b86bd66f95b470
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sat Jun 24 10:59:48 2017 +0200

    json data store: allow name_id in edit/delete methods (#17171)
#8

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF