Projet

Général

Profil

Development #30543

auth_saml: prendre en compte le paramètre request dans "authenticate"

Ajouté par Serghei Mihai il y a environ 5 ans. Mis à jour il y a environ 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
12 février 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Suite à #30541.


Fichiers


Demandes liées

Lié à django-mellon - Development #30541: django1.11: authenticate() got an unexpected keyword argument 'request'Fermé12 février 2019

Actions

Révisions associées

Révision 506df5b5 (diff)
Ajouté par Serghei Mihai il y a environ 5 ans

auth_saml: remove useless credentials param and add optional request (#30543)

And so assure compliance with Django 1.11

Historique

#1

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

#2

Mis à jour par Benjamin Dauvergne il y a environ 5 ans

Tu peux virer credentials.

#3

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

Oui, parce que Django 1.11 fait un inspect des paramètres de la fonction authenticate et ne passe pas request s'il n'est pas attendu.

#4

Mis à jour par Benjamin Dauvergne il y a environ 5 ans

Tu peux ajouter request quand même.... le jour où django-mellon s'en servira on sera bien content de ne pas revenir sur auth-saml.

#6

Mis à jour par Benjamin Dauvergne il y a environ 5 ans

Serghei Mihai a écrit :

ok.

Dans l'appel à super(), (saml_attributes=saml_attributes, request=request).

#8

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

  • Lié à Development #30541: django1.11: authenticate() got an unexpected keyword argument 'request' ajouté
#9

Mis à jour par Benjamin Dauvergne il y a environ 5 ans

  • Statut changé de Solution proposée à Solution validée

Ack.

#10

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit 506df5b5ce90303996184546fdbf51c86af7035c (origin/master, origin/HEAD)
Author: Serghei Mihai <smihai@entrouvert.com>
Date:   Tue Feb 12 10:00:31 2019 +0100

    auth_saml: remove useless credentials param and add optional request (#30543)

    And so assure compliance with Django 1.11
#11

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

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF