Bug #7453
hobo: déploiement avec base_url se terminant par un /
0%
Description
Ça crée des URL avec des double slash pour les metadata, ça crée pas un instance path correct, etc.
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a presque 9 ans
- Fichier 0001-check_hobos-base_url-may-end-with-a-7453.patch 0001-check_hobos-base_url-may-end-with-a-7453.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Je pense qu'il suffit de corriger l'instance_path, le frontoffice_url et les URL des metadata. Voici ma proposition dans check_hobo.
(il y a par ailleurs du nettoyage à faire dans la sortie hobo.json, dans hobo)
Mis à jour par Frédéric Péters il y a presque 9 ans
Je n'aurais pas d'unilatéral :
metadata_url = '%sidp/saml2/metadata' % idp['base_url']
Dans la mesure où il peut rester des hobo.json sans / final.
Le "get_instance_path" était déjà bien faux avant, l'est toujours, en fait le '=' devrait être un '+=', pour ne pas perdre le nom de domaine, et ça devrait être '+%s' et non '%s+' (le résultat qu'on veut pour "https://teleformulaires.pratic59.fr/landrecies/", c'est "teleformulaires.pratic59.fr+landrecies"). On peut aussi se dire que c'est un bug différent et le corriger indépendamment.
Mis à jour par Thomas Noël il y a presque 9 ans
- Fichier 0001-check_hobos-base_url-may-end-with-a-7453.patch 0001-check_hobos-base_url-may-end-with-a-7453.patch ajouté
Frédéric Péters a écrit :
Je n'aurais pas d'unilatéral :
metadata_url = '%sidp/saml2/metadata' % idp['base_url']
Dans la mesure où il peut rester des hobo.json sans / final.
Et aussi dans le calcul des MD du sp, donc. Voici un autre patch.
Le "get_instance_path" était déjà bien faux avant, l'est toujours, en fait le '=' devrait être un '+=', pour ne pas perdre le nom de domaine, et ça devrait être '+%s' et non '%s+' (le résultat qu'on veut pour "https://teleformulaires.pratic59.fr/landrecies/", c'est "teleformulaires.pratic59.fr+landrecies"). On peut aussi se dire que c'est un bug différent et le corriger indépendamment.
Oui, un autre ticket, je préfère.
Mis à jour par Frédéric Péters il y a presque 9 ans
Ce patch ne m'enthousiasme guère et j'y préfère encore le fix moche mais plus court que j'avais posé sur le serveur :
@@ -67,6 +67,8 @@ class CmdCheckHobos(Command):
service = [x for x in self.all_services.get('services', []) if \
x.get('service-id') == 'wcs' and x.get('base_url') == base_url][0]
+ if base_url.endswith('/'): # wcs doesn't expect a trailing slash
+ service['base_url'] = base_url[:-1]
pub.app_dir = os.path.join(global_app_dir,
self.get_instance_path(service))
?
Mis à jour par Thomas Noël il y a presque 9 ans
Quitte à faire moche, autant que ça soit discret, effectivement. Je pousse donc plutôt ça.
Mis à jour par Thomas Noël il y a presque 9 ans
- Statut changé de En cours à Résolu (à déployer)
commit eaf5772cef8e2097740c49a02453175ab73d754b
Author: Thomas NOEL <tnoel@entrouvert.com>
Date: Mon Jun 8 10:05:55 2015 +0200
check_hobos: base_url may end with a / (#7453)
(sorry pour l'attribution, j'ai fait vite)
check_hobos: base_url may end with a / (#7453)