Produits Entr'ouvert » django-mellon

À voir si c'est vraiment plus intéressant de complexifier django-mellon ou de juste écrire un adapter pour docbow, qu'en penses-tu ?

Tu peux complètement remplacer Adapter.lookup_user() pour y faire ce que tu veux exactement sans te préoccuper d'éventuels changements dans django-mellon ou du fait que la configuration soit bien faite (si on est au pfwb c'est comme ci, si pw c'est comme ça).

À la rigueur, histoire de pas perdre le peu de debug qu'on a déjà sur ce point :

def check_field_path(base_klass, field_path):
    name = field_path.split('__', 1)[0]
    rest = field_path.split('__', 1)[1:]
    try:
        field = base_klass._meta.get_field(name)
    except FieldDoesNotExist:
        return False
    if rest:
        if not field.is_relation:
            return False        
        return check_field_path(field.related_model, rest[0])
....
    if not check_field_path(User, user_field):
        logger.error('invalid LOOKUP_BY_ATTRIBUTES configuration %r, user field %s does not exist',
                     line, user_field)

        continue

Benjamin Dauvergne a écrit :

À voir si c'est vraiment plus intéressant de complexifier django-mellon ou de juste écrire un adapter pour docbow, qu'en penses-tu ?

En fait oui, c'est nettement plus simple, je préfère laisser tomber mon idée initiale.

Par contre pour ne pas avoir à tout re-écrire trop de choses dans un adapter spécifique, un petit hook du genre :

         if lookup_by_attributes:
             user = self._lookup_by_attributes(idp, saml_attributes, lookup_by_attributes)

+        if user is None and hasattr(self, 'custom_lookup'):
+            user  = self.cutom_lookup(idp, saml_attributes)
+

Ça t'irait ?

Hmmm idéalement il faudrait découper lookup_user qui en fait trop (ça fait lookup_by_nameid() or (lookup_by_attribute() and link) or create()).

Je réécrirait plutôt ce bout de code :

...
        user = None
        lookup_by_attributes = utils.get_setting(idp, 'LOOKUP_BY_ATTRIBUTES')
        if lookup_by_attributes:
            user = self._lookup_by_attributes(idp, saml_attributes, lookup_by_attributes)
...

comme cela :

    def lookup_user_by_attributes(self, idp, saml_attributes):
        rules = utils.get_setting(idp, 'LOOKUP_BY_ATTRIBUTES'):
        if rules:
        ....
        return None

...
    def lookup_user(...):
....
        user = self.lookup_user_by_attributes(idp, saml_attributes)
...

T'auras plus qu'à surcharger ça et si ça renvoie None appliquer tes propres règles, wadoyouthink ?