Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Configuration centralisée optionnelle ? #27

Open
camillemonchicourt opened this issue Nov 6, 2020 · 2 comments
Open

Configuration centralisée optionnelle ? #27

camillemonchicourt opened this issue Nov 6, 2020 · 2 comments
Labels
enhancement New feature or request question Further information is requested

Comments

@camillemonchicourt
Copy link
Member

@amandine-sahl a remonté un problème de synchronisation des données qui semblait applicatif : #26

En fait celui-ci est du fait que la configuration centralisée n'avait pas été mise en place sur son serveur GeoNature.
En effet celle-ci est indiquée comme optionnelle dans la documentation.
Alors qu'en fait si la table gn_commons.t_mobile_apps n'est pas renseignée, la route api/gn_commons/t_mobile_apps renvoie une erreur 404 et cela fait planter la synchronisation des données.
De la même manière, si un fichier de configuration n'est pas disponible sur le serveur GeoNature, alors cette même route api/gn_commons/t_mobile_apps renvoie une internal error (500) et cela fait aussi planter la synchronisation des données.

Plusieurs possibilités :

  • Côté GeoNature, faire en sorte que la route ne soit pas en 404 ni 500 quand les infos ne sont pas renseignés
  • Imposer la centralisation
  • Faire en sorte que la synchro fonctionne même si la route de centralisation ne renvoie rien ou est en échec
  • Mettre un message plus clair quand la route ne fonctionne pas
  • Ajouter des tests et des vérifications des erreurs dans l'application Sync-mobile

Il me semble dommage d'imposer la configuration centralisée.
Ça peut être gênant d'imposer les mêmes paramètres à tous les utilisateurs d'une instance (chemin des fichiers carto, étendue spatiale...) dans des contextes associatifs par exemple.

Pour commencer :

  • Je vais essayer de revoir la documentation pour indiquer qu'il faut vérifier le contenu de la route de configuration centralisée
  • On va voir côté GeoNature si on peut éviter la 404 quand la table gn_commons.t_mobile_apps n'est pas renseignée

Mais il serait utile d'avoir des vérifications et un message au niveau de Sync-mobile quand elle rencontre un soucis avec la route.

Et dans un second temps, voir si on peut imaginer un fonctionnement opérationnel sans configuration centralisée.

@camillemonchicourt
Copy link
Member Author

C'est pas idéal, mais pour commencer rapidement, j'ai fait un ajustement dans la doc indiquant que la conf centralisée n'est actuellement pas optionnelle mais nécessaire au fonctionnement de la synchronisation : PnX-SI/gn_mobile_occtax@4f0a00b

@camillemonchicourt camillemonchicourt added enhancement New feature or request question Further information is requested labels Feb 6, 2021
@DonovanMaillard
Copy link
Collaborator

DonovanMaillard commented Aug 4, 2021

A reconsidérer avec l'option suivante vers laquelle on prévoit de se tourner prochainement :

  • Configuration centralisée obligatoire (permet un déploiement plus simple et une gestion de la conf sans intervenir sur chaque terminal)
  • Peut être surcouchée par une configuration locale pour les utilisateurs avancés ou les besoins particuliers

Cf : PnX-SI/gn_mobile_occtax#95

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants