Produits Entr'ouvert » Authentic 2

Je me permet de pointer que vraiment vraiment ça aiderait des gens à travailler convenablement avec Publik.

Mais où renvoyer si l'agent fait cancel ? Il faut prévoir un cancel_url aussi.

La double accolade donne trop l'idée qu'on pourrait faire n'importe quoi comme gabarit.

J'aime bien l'idée d'un ou d'une suite de caractères de fermeture.

(Si par exemple à l'avenir on veut des URL du genre /{{user_ou}}/{{user}}/)

D'ailleurs, ça me fait penser que dans select_next_url on pourrait tester qu'aucune des valeurs du dico replace ne contient une autre clé, du genre

for value in replace.values():
    for key in replace.keys():
        if key in value and replace[key] != value
            raise ValueError…

Et tester, de façon similaire, qu'aucune des clés n'en contient une autre, sinon on est pareillement fichu.

Edit: Ce qui me conforte dans l'idée d'avoir un ou des caractères de fermeture.

Frédéric Péters a écrit :

La double accolade donne trop l'idée qu'on pourrait faire n'importe quoi comme gabarit.

Oui c'est exactement pour ça que j'ai mis $XXXX aussi parce que ça n'interpolle pas juste la chaîne ça la quote en plus (pas possible d'avoir un slash d'URL dedans).

Paul Marillonnet a écrit :

D'ailleurs, ça me fait penser que dans select_next_url on pourrait tester qu'aucune des valeurs du dico replace ne contient une autre clé, du genre
[...]

Non, si ton UUID c'est "$UUID" et bien soit, c'est con, mais c'est ton UUID (pour l'instant UUID est un champ texte libre, dans le futur ça deviendra peut-être une colonne UUID native postgres et ça ne passera plus).

Et tester, de façon similaire, qu'aucune des clés n'en contient une autre, sinon on est pareillement fichu.

Foutu de quoi ? Le remplacement n'est pas récursif et pour l'instant il n'y a qu'un remplacement permis, $UUID, et je serai très frileux d'en avoir d'autres à l'avenir; l'intérêt d'avoir un identifiant global unique c'est ne pas en avoir besoin d'un autre (et si un logiciel veux renvoyer de /$UUID/ vers /{{user.ou.slug}}/$UUID et bien il se débrouillera pour le faire, a priori si on lui envoie l'$UUID c'est qu'il a accès à l'API ou a une copie locale de l'annuaire).

Et les $ sont interdits dans les slugs.

Je ne changerai rien.

Ok, va comme ça.