Projet

Général

Profil

Bug #8686

Permettre de fonctionner en multitenant sur un port non standard

Ajouté par Frédéric Péters il y a plus de 8 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
18 octobre 2015
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Pour le moment quand hobo est en multitenant on est obligé de tourner sur le port 80 ou 443.

Dans #8170 on a ajouté ça :

+    def is_secure(self):
+        if os.path.exists(os.path.join(self.get_directory(), 'unsecure')):
+            return False
+        return True
+
+    def get_base_url(self):
+        if self.is_secure():
+            return 'https://%s' % self.domain_url
+        else:
+            return 'http://%s' % self.domain_url

Le patch attaché ajoute la possibilité de mettre le base_url souhaité dans un fichier base_url du tenant.

On pourrait aussi discuter de retirer le is_secure() pour ne pas avoir deux stat().


Fichiers

Révisions associées

Révision a0621508 (diff)
Ajouté par Frédéric Péters il y a plus de 8 ans

multitenant: support a base_url to get an alternate base_url (#8686)

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 8 ans

#2

Mis à jour par Thomas Noël il y a plus de 8 ans

Ok. Mais oui, j'en profiterai bien pour virer le is_secure(), oui (ceux qui veulent du http utilisent base_url)

#3

Mis à jour par Frédéric Péters il y a plus de 8 ans

  • Statut changé de En cours à Résolu (à déployer)
commit a062150823253b71215c9eb8c6eb58eb241a35a6
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sun Oct 18 18:22:31 2015 +0200

    multitenant: support a base_url to get an alternate base_url (#8686)

Pour la suppression du is_secure, je crée un nouveau ticket : #9539.

#4

Mis à jour par Frédéric Péters il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF