Project

General

Profile

Development #40350

définir un template de base pour authentic2_auth_saml

Added by Serghei Mihai 9 months ago. Updated 5 months ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Start date:
02 Mar 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Depuis #35083 on peut déclarer le template de base pour mellon.
Il faudrait s'en servir pour éviter les erreurs 500 lors d'un souci de SSO.

0001-auth_saml-define-templates-base-page-40350.patch View (790 Bytes) Serghei Mihai, 22 Jun 2020 02:30 PM

0001-auth_saml-define-templates-base-page-40350.patch View (780 Bytes) Serghei Mihai, 30 Jun 2020 04:46 PM


Related issues

Related to django-mellon - Development #35083: Pouvoir indiquer un préfixe particulier pour les templates internes à mellon Solution déployée 26 Jul 2019

Associated revisions

Revision d55eddcd (diff)
Added by Serghei Mihai 5 months ago

auth_saml: define templates base page (#40350)

History

#1 Updated by Serghei Mihai 9 months ago

  • Related to Development #35083: Pouvoir indiquer un préfixe particulier pour les templates internes à mellon added

#2 Updated by Benjamin Dauvergne 5 months ago

  • Assignee set to Serghei Mihai

#3 Updated by Serghei Mihai 5 months ago

#4 Updated by Valentin Deniaud 5 months ago

Le patch ne s'applique pas, il a l'air basé sur des modifs que tu avais en local. Pousse une branche ?

#5 Updated by Serghei Mihai 5 months ago

Yep, micmac chez moi en local.

Patch correct et branche poussée.

#6 Updated by Benjamin Dauvergne 5 months ago

C'est l'occasion d'ajouter un test qui échoue pour voir si le template est bien pris en compte et augmenter joyeusement la couverture du code, c'est dans tests/test_auth_saml.py.

#7 Updated by Serghei Mihai 5 months ago

  • Status changed from Solution proposée to En cours

Ouep, mais je doute fort que ça augmente la couverture du code d'authentic vu que l'argument s'applique à la vue de mellon.

#8 Updated by Benjamin Dauvergne 5 months ago

La couverture est une notion abstraite c'est pas seulement le chiffre renvoyé par coverage, faut juste voir qu'on arrive à traitre une erreur django-mellon sans planter sur le rendu du template.

#9 Updated by Benjamin Dauvergne 5 months ago

  • Status changed from En cours to Solution validée

Allez c'est bon.

#10 Updated by Benjamin Dauvergne 5 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit e992567dab3256843e8876678beeb9bf0972d748
Author: Serghei Mihai <smihai@entrouvert.com>
Date:   Tue Jun 30 16:45:23 2020 +0200

    auth_saml: define templates base page (#40350)

#11 Updated by Frédéric Péters 5 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF