Produits Entr'ouvert » Mandaye

Pourquoi dans un if? Il se peut que username soit vide?

La regex (?P<username>[\w+]*) devrait être (?P<username>\w+).

Or la longueur maximale du username django est de 30 caractères

Ce n'est pas le bout le plus intéressant de l'histoire. Le bout intéressant qu'il faut documenter sous forme de commentaire au niveau du code c'est que le provisionning assuré par hobo tronque l'uuid à 30 caractères.

J'allais aussi ajouter qu'il ne fallait sans doute pas préfixer le message avec "archimed" pour du code dans mandayejs/views.py mais j'ai malheureusement ouvert ce fichier.

Grosse bourde, dans ce patch. J'en refais un autre.

Hobo n'assure pas de provisionning, les utilisateurs sont créés lors du SSO.

Uh uh uh ? D'une part ça serait un bug, il doit y avoir un agent hobo pour le provisionning des usagers. D'autre part la coupure à 30 caractères reste le fait d'un composant externe (si c'est le SSO, c'est mellon), et ça doit être noté dans un commentaire plutôt qu'avoir un [:30] qui vient sans explication.

Et l'absence de provisionning par Hobo, une réaction ?

        # mellon truncates username to 30 characters
        # thus the passed username should be truncated to 30 characters
        # for searching purposer

Se relire et corriger.

            user = User.objects.get(username=username)
            # *startswith* to deal with the looseness of sqlite
            # and the strictness of postgres
            user = User.objects.get(username__startswith=username)

Uh uh uh non; ça ne va pas de dire "pas la moindre idée de ce qu'il y a dans la base, tapons dedans"; que ça soit sqlite ou postgresql la même info doit être stockée dedans.

Uh uh uh ? D'une part ça serait un bug, il doit y avoir un agent hobo pour le provisionning des usagers

Je me trompe peut etre, mais je ne vois pas trop l'interet du provisionning hobo pour mandayejs. Est ce qu'il y'a un interet a avoir des utilisateurs dans mandayejs sans que ceux ci ne soient liés ? Je vois plutot MandayeJS comme Passerelle, ou les seuls utilisateurs qui y sont créés sont ceux qui s'y sont déja connecté.

comme Passerelle, ou les seuls utilisateurs qui y sont créés sont ceux qui s'y sont déja connecté

Uh uh uh. Notre système de provisionning doit fonctionner sur tous nos modules, si tu vois que ce n'est pas le cas pour Passerelle, il faut soulever un bug.

Se relire et corriger.

Ok, merci. C'est fait.

Mais le patch que tu attaches à nouveau, c'est l'autre ? (et pourquoi deux patchs, d'ailleurs)

Il y a des virgules dans le name_id transmis?

En fait le [\w,-] est une erreur qu'on retrouve souvent dans passerelle, et qui a commencé à se répandre dans combo (une fois dans calendar/urls.py). A ne pas trop répéter :)

Ack

Dans #17196 on découvre des uuid qui contiennent d'autres caractères.