Produits Entr'ouvert » Chrono

À la réception itérer dans KNOWN_SERVICES et si service connu modifier l'URL pour commencer par {{whatever_url}}; attention alors à échapper correctement si qui dans l'URL initiale aurait pu ressembler à un gabarit. Ou faire sans gabarit et ad hoc, remplacer par genre publik://whatever/le chemin/ et dans requests_wrapper si le protocole est publik:// chercher dans KNOWN_SERVICES pour faire la correspondance.

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

remplacer par genre publik://whatever/le chemin/ et dans requests_wrapper si le protocole est publik:// chercher dans KNOWN_SERVICES pour faire la correspondance.

J'aime bien cette idée.

Aussi je ne me lance pas dans une migration, ça pourra se faire au cas par cas si besoin.

Emmanuel Cazenave a écrit :

Aussi je ne me lance pas dans une migration, ça pourra se faire au cas par cas si besoin.

• une migration pourrait prendre de trèèès longues minutes (voire pire vu les volumes sur certains sites)
• or elle n'est pas obligatoire, les deux formes d'URL fonctionnent sans soucis (ça n'est en réalité utile que pour aider à un changement d'URL de site, encore rare)
• au fur et à mesure les rendez-vous en format historique vont disparaitre

Bref, effectivement, ça ne vaut pas la peine.

Et si on juge un jour que si, c'est nécessaire, alors ça pourrait tout à fait être ajouté en dehors de ce patch, avec une commande spécifique qui ferait la modif des url, qui s'exécuterait alors que le service chrono est déjà démarré ; pas de downtime, c'est mieux.

Relecture après autre causerie au bureau : je trouve que le «translate_url» n'est effectivement pas très explicite et arrive un peu au milieu de nulle part.

Est-ce qu'on ne serait pas plus explicite à créer un module chrono/utils/publik.py avec deux fonctions translate_to_publik_url() et translate_from_publik_url() ... Et ces fonctions pourraient commencer par :

function translate_to_publik_url(url):
   known_services = getattr(settings, 'KNOWN_SERVICES', None)
   if not known_services:
       return url
   ...

afin qu'on pige bien que cette affaire dépend d'un éventuel settings.KNOWN_SERVICES ?

(Bien sûr mon idée derrière ça est que chrono/utils/publik.py soit le même partout et finisse pas trouver son chemin vers hobo.utils ... ah, les rêves)

(je préférerais laisser urls.py aux fichiers urls.py de routing de django)

Je trouverais le diff serait plus joli si

-        cancel_callback_url=payload.get('cancel_callback_url', ''),
+        cancel_callback_url=translate_to_publik_url(payload.get('backoffice_url')),

(et il y a déjà get_event_detail où c'est fait comme ça)

Emmanuel Cazenave a écrit :

Valentin Deniaud a écrit :

Je trouverais le diff serait plus joli si

Fait.

Un saut de ligne résiduel ligne 648 empêche ce patch d'atteindre la perfection.