Projet

Général

Profil

Bug #14813

Dans le cas d'un retour JSON érroné par PhantomJS, l'utilisateur est quand meme associé

Ajouté par Josué Kouka il y a environ 7 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Josué Kouka
Catégorie:
-
Version cible:
-
Début:
30 janvier 2017
Echéance:
% réalisé:

100%

Temps estimé:
Patch proposed:
Oui
Planning:

Fichiers

0001-don-t-link-user-if-PhantomJS-response-is-erroneous-1.patch (1,44 ko) 0001-don-t-link-user-if-PhantomJS-response-is-erroneous-1.patch Josué Kouka, 30 janvier 2017 15:48
0001-don-t-associate-user-if-PhantomJS-response-is-errone.patch (3,53 ko) 0001-don-t-associate-user-if-PhantomJS-response-is-errone.patch Josué Kouka, 31 janvier 2017 16:46
0001-don-t-associate-user-if-PhantomJS-response-is-errone.patch (3,72 ko) 0001-don-t-associate-user-if-PhantomJS-response-is-errone.patch Josué Kouka, 31 janvier 2017 18:05
0001-don-t-associate-user-if-PhantomJS-response-is-errone.patch (2,13 ko) 0001-don-t-associate-user-if-PhantomJS-response-is-errone.patch Josué Kouka, 06 février 2017 10:32
0002-test-improve-json-response-error-14813.patch (2,38 ko) 0002-test-improve-json-response-error-14813.patch Josué Kouka, 06 février 2017 10:32

Révisions associées

Révision 9d1f6af5 (diff)
Ajouté par Josué Kouka il y a environ 7 ans

don't associate user if PhantomJS response is erroneous (#14813)

Révision 905c44d0 (diff)
Ajouté par Josué Kouka il y a environ 7 ans

test: improve json response error (#14813)

Historique

#1

Mis à jour par Josué Kouka il y a environ 7 ans

#2

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

Mmhhh non, il ne faut pas utiliser "failure" qui va déclencer un credentials.delete().

#4

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

  • logger.error(stdout) : on va pas comprendre dont il s'agit, il faut ajouter une petite info qui signale "json incorrect"
  • j'aurai mis le "elif result.get('result') == 'json_error':" au dessus du cas redirect, c-a-d groupé avec "failure" et "timeout", les erreurs quoi.
#5

Mis à jour par Serghei Mihai il y a environ 7 ans 

_('invalid response from server')

avec un "I" majuscule.

#6

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

Serghei Mihai a écrit :

[...]

avec un "I" majuscule.

Heu non, les autres messages sont de la même forme : wrong user credentials / server took too long to respond

#8

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

Pourquoi le test change-t-il tant ?

#9

Mis à jour par Josué Kouka il y a environ 7 ans

Thomas Noël a écrit :

Pourquoi le test change-t-il tant ?

J'ai trouvé que c'était mieux de tester le circuit complet plutot qu'une seule fonction. Je voulais m'assurer que le bon message d'erreur était remonté à l'utilisateur.

#10

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

Oui mais non, alleÿ, deux patches alors.

#12

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

Ack

#13

Mis à jour par Josué Kouka il y a environ 7 ans

  • Statut changé de En cours à Résolu (à déployer)
  • % réalisé changé de 0 à 100
#14

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

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

Formats disponibles : Atom PDF