Project

General

Profile

Tâche #21453

Scénario #21866: Traitement express MEN (46-48)

Internal Server Error, si user non trouvé en local lors d'une fédération

Added by Christophe LEON over 5 years ago. Updated over 5 years ago.

Status:
Fermé
Priority:
Normal
Assigned To:
Emmanuel GARETTE
Target version:
sprint 2017 46-48 Equipe MENSR
Start date:
09/26/2017
Due date:
% Done:

100%

Estimated time:
3.00 h
Spent time:
2.25 h
Remaining (hours):
0.0

Description

Lors d'une fédération si le user n'est pas trouvé en local , il y a le message d'erreur 'Internal Server Error' au lieu du message habituel 'Fédération : Les informations reçues ne permettent pas de vous identifier'

En cause /usr/share/sso/saml_resources.py vers la ligne 791, il y a 

resp = gen_page_err(content, return_url, css)
    return self.set_headers(gen_page_err)

Cela devrait être

resp = gen_page_err(content, return_url, css)
    return self.set_headers(resp)

Associated revisions

Revision ab009059 (diff)
Added by Emmanuel GARETTE over 5 years ago

Renvoyer la réponse et non une fonction si un user est non trouvé en local lors d'une fédération

ref #21453 @1h

History

#1 Updated by Joël Cuissinat over 5 years ago

  • Tracker changed from Anomalie to Demande

#2 Updated by Joël Cuissinat over 5 years ago

  • Tracker changed from Demande to Tâche
  • Estimated time set to 3.00 h
  • Parent task set to #21866
  • Remaining (hours) set to 3.0

Je suppose qu'on peut appliquer la correction presque "à l'aveugle" :)

Sur quelle version d'EOLE vous êtes impactés ?

#3 Updated by Emmanuel GARETTE over 5 years ago

  • Status changed from Nouveau to En cours
  • Assigned To set to Emmanuel GARETTE

#4 Updated by Emmanuel GARETTE over 5 years ago

  • % Done changed from 0 to 80
  • Remaining (hours) changed from 3.0 to 1.0

En regardant le commit eole-sso:f7b1083c il est évident qu'il faut retourner la réponse et non la fonction.

Le problème existe depuis 2.4.0 et sur toutes les versions maintenues.

J'ai créé une branche à partir de 2.5.0 et je n'ai mergé que dans 2.6.2.

Voir si on envisage le backport.

#5 Updated by Emmanuel GARETTE over 5 years ago

  • Remaining (hours) changed from 1.0 to 0.25

Backport réalisé en 2.5.2 et 2.6.1, journaux mis à jour.

#6 Updated by Emmanuel GARETTE over 5 years ago

  • % Done changed from 80 to 100

#7 Updated by Scrum Master over 5 years ago

  • Project changed from EoleSSO to Distribution EOLE
  • Status changed from En cours to Résolu

#8 Updated by Benjamin Bohard over 5 years ago

  • Remaining (hours) changed from 0.25 to 0.0

#9 Updated by Benjamin Bohard over 5 years ago

  • Status changed from Résolu to Fermé

Also available in: Atom PDF