Development #64344
Wrapper requests : interpolation dans les noms de domaine sur la base des legacy_urls
0%
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Emmanuel Cazenave il y a presque 2 ans
- Lié à Support #57729: Migration d'instances de Publik : changement de nom de domaine ajouté
Mis à jour par Emmanuel Cazenave il y a presque 2 ans
- Fichier 0001-misc-use-legacy-urls-to-call-up-to-date-urls-64344.patch 0001-misc-use-legacy-urls-to-call-up-to-date-urls-64344.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
En se basant sur #64347.
Mis à jour par Thomas Noël il y a presque 2 ans
Perso je suis pas fan des getattr sur settings : je préférerais un LEGACY_URLS_MAPPING = None dans le settings.py, avec au dessus un petit commentaire qui explique ce que ça peut contenir (un dictionnaire ancien:nouveau des netloc des URLs).
Mais comme on a déjà des getattr(settings...) pour d'autres choses du même ordre, bon.
Ceci étant, pourquoi ne pas intégrer cette gestion dans translate_from_publik_url() ? Parce que là, tu as joué dans requests_wrapper.py, mais translate_from_publik_url est également utilisé pour construire les URL dans la classe Booking. Il me semble que ça serait nécessaire de gérer les LEGACY_URLS_MAPPING là aussi, non ?
Mis à jour par Emmanuel Cazenave il y a presque 2 ans
- Fichier 0001-misc-use-legacy-urls-to-call-up-to-date-urls-64344.patch 0001-misc-use-legacy-urls-to-call-up-to-date-urls-64344.patch ajouté
Thomas Noël a écrit :
Ceci étant, pourquoi ne pas intégrer cette gestion dans
translate_from_publik_url()?
Effectivement, bien vu.
Mis à jour par Thomas Noël il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Emmanuel Cazenave il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 1ca5e77bd572a5187b409cbc0366a9aafb21ed35
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date: Wed Apr 20 17:50:57 2022 +0200
misc: use legacy urls to call up to date urls (#64344)
Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: use legacy urls to call up to date urls (#64344)