Projet

Général

Profil

Development #23362

Ajouter mellon à INSTALLED_APPS dès les settings de base

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Catégorie:
-
Version cible:
-
Début:
24 avril 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Pour le moment c'est ajouté via debian/debian_config.py et tests/settings.py.


Fichiers

Révisions associées

Révision 8d96bf96 (diff)
Ajouté par Frédéric Péters il y a environ 6 ans

general: always declare django-mellon if it's available (#23362)

Historique

#1

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

#2

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

Oui mais on fait ça partout, la doc de hobo le dit d'ailleurs :

$ head debian/debian_config_common.py 
# This file is sourced by "execfile" from django settings

# Example of django settings.py:
#
# (...)
# PROJECT_NAME = 'name'
# INSTALLED_APPS += ('mellon',)   # SAML2 authentication
# execfile('/usr/lib/hoho/debian_config_common.py')
# execfile('/etc/%s/settings.py' % PROJECT_NAME)

J'aimerai retrouver la raison de l'affaire.

#3

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

Je ne sais plus pourquoi ça a été fait comme ça mais avec 1.11 il est devenu plus important d'avoir les applications vraiment déclarées dans INSTALLED_APPS (sinon ça dit qu'il faut déclarer un app_label etc.).

#4

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

Juste pour ma compréhension : le warning apparait parce qu'on se retrouve éventuellement avec mellon référencé dans les AUTHENTICATION_BACKENDS et pas dans INSTALLED_APPS ?

En tout cas le patch proposé me semble correct. La seule implication est que mellon ne serait plus à la même position dans les INSTALLED_APPS ; je ne pense pas que ça joue.

Ack.

#5

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

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

Je n'ai plus la situation précise en tête, c'était il y a quelques jours; en gros sur l'exécution d'une commande de management sans trop de rapport (genre un makemessages?) j'obtenais le message sur l'app_label.

commit 8d96bf9656cf2574368cce9a938715f83bc71f1d
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Apr 24 10:09:10 2018 +0200

    general: always declare django-mellon if it's available (#23362)
#6

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

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

Formats disponibles : Atom PDF