Simplification des paramètres URL dans la config mviewerstudio

Gaetanbrl commented 1 year ago

Lié à #158

Description

Dans mviewerstudio, plusieurs configurations permettent de faire fonctionner le système de sauvegarde / preview / listing.

Configuration frontend

fichier mviewerstudio_backend/static/apps/config.json :

Configuration backend

fichier mviewerstudio_backend/settings.py :

    CONF_PATH_FROM_MVIEWER = os.getenv("CONF_PATH_FROM_MVIEWER", "apps/store/")
    EXPORT_CONF_FOLDER = os.getenv("EXPORT_CONF_FOLDER", "./store")

Question

Je trouve ce paramétrage un peu complexe.

On a des paramètres dans le frontend qui ne sont pas utilisés (e.g CONF_PATH_FROM_MVIEWER)
Le backend réalise des modifications pour fournir un coup /store/maconfig et un coup /maconfif selon que l'on serve la liste des configuration ou l'emplacement du fichier
Le frontend réalise des contrôles un peu comples pour charger le XML, notamment avec le paramètre mviewer_short_url.

Quelles est la + value des URLs courtes ?

Aussi, je me demande si ces seuls paramètres ne sont pas suffisants :

backend - CONF_PATH_FROM_MVIEWERDevrait être seulement un paramètre du front pour la redirection vers le mviewer
frontend - mviewer_instance pour cibler le bon mviewer
backend - EXPORT_CONF_FOLDER- pour bien localiser / sauvegarder les configs

spelhate commented 1 year ago

On a des paramètres dans le frontend qui ne sont pas utilisés (e.g CONF_PATH_FROM_MVIEWER)

A l'origine, tous les paramètres (back + front étaient gérés dans un seul fichier). Tu proposes de séparer les choses @Gaetanbrl ?

Quelles est la + value des URLs courtes ?

Il faut poser la question à @bchartier car il me semble que ça a été développé par neogeo

Aussi, je me demande si ces seuls paramètres ne sont pas suffisants :

backend - CONF_PATH_FROM_MVIEWER Devrait être seulement un paramètre du front pour la redirection vers le mviewer

KO. Le backend en a également besoin pour générer les URL vers les fichiers XML (https://github.com/mviewer/mviewerstudio/tree/master/srv#srvlistphp)

frontend - mviewer_instance pour cibler le bon mviewer

Ok

backend - EXPORT_CONF_FOLDER - pour bien localiser / sauvegarder les configs

Ok, mais actuellement c'est uniquement repris par le backend php.

Gaetanbrl commented 1 year ago

Le backend en a également besoin pour générer les URL vers les fichiers XML

qui consomme ces URLs ?

Je précise, cette issue est en lien avec un éventuel passage backend Python et suppression PHP.

Gaetanbrl commented 1 year ago

Autre possibilité (évolution + analyse), Trouver un moyen de centraliser la configuration et éviter 2 fichiers avec des config qui semblent redondantes.

mviewer / mviewerstudio