Produits Entr'ouvert » django-mellon

Pourquoi pas, mais ça corrige pas vraiment le bogue, les metadata seront foireuses à un moment.

Je rapatrie ma question de #26141 :
Je me demande dans quel cadre un SP SAML peut avoir des metadata sans clé publique ?
Auquel cas je préférerais que mellon réponde 404 quand app_settings.PUBLIC_KEYS est vide.
Quitte à gérer un possible "settings.METADATA_WITHOUT_PUBLIC_KEYS_IS_OK_GO_GO_GO == True" dans les settings (et 404 si cet attribut est absent)

Thomas Noël a écrit :

Pourquoi pas, mais ça corrige pas vraiment le bogue, les metadata seront foireuses à un moment.

On passerait quand même de "foireuse de façon permanente (jusqu'à redémarrage dur serveur)" à "très brièvement foireuse", ça me semble être une avancée.

Statut changé de Nouveau à Solution validée

Comme ce ticket partait sur autre chose, j'ai déplacé le patch vers #13881.

(pas plus de discussion de ma part)

Je serai pour fermer ce ticket et voir comment rendre l'initialisation d'un tenant plus atomique, dans cette partie:

            tenant_dir = os.path.join(tenant_base, hostname)
            if os.path.exists(tenant_dir):
                raise CommandError('tenant already exists')
            tenant_dir_tmp = os.path.join(
                    tenant_base, hostname + '__CREATION_IN_PROGRESS.invalid')
            try:
                os.mkdir(tenant_dir_tmp, 0755)
            except OSError as e:
                raise CommandError('cannot start tenant creation (%s)' % str(e))
            # tenant creation in database
            try:
                connection.set_schema_to_public()
                schema = TenantMiddleware.hostname2schema(hostname)
                tenant = get_tenant_model()(schema_name=schema, domain_url=hostname)
                if verbosity >= 1:
                    print
                    print self.style.NOTICE("=== Creating schema ") \
                        + self.style.SQL_TABLE(tenant.schema_name)
                with transaction.atomic():
                    tenant.create_schema(check_if_exists=True)
            except Exception as e:
                os.rmdir(tenant_dir_tmp)
                raise CommandError('tenant creation failed (%s)' % str(e))
            for folder in ('media', 'static', 'templates'):
                path = os.path.join(tenant_dir_tmp, folder)
                os.mkdir(path, 0755)
            # activate the tenant by creating its directory
            os.rename(tenant_dir_tmp, tenant_dir)

J'ai déjà signalé dans un autre ticket que la création du schéma devrait se faire dans une transaction (en cas d'échec sur l'initialisation du filesystem, on se retrouve avec un schéma en vrac, idem en cas de souci sur l'application des migrations, schéma incomplet). Actuellement c'est le répertoire qui fait foi quand à l'existence d'un tenant.