[Minor] Remove external data servers and rename local one

assemblee-virtuelle / archipelago

Fostering interconnections between communities by creating synergies between their platforms

Apache License 2.0

14 stars 6 forks source link

[Minor] Remove external data servers and rename local one #105

Closed mguihal closed 11 months ago

mguihal commented 1 year ago

Hello,

Je ne connais pas l'historique du repo, mais je m'interroge sur le fait d'avoir des serveurs externes en dur dans ce fichier dans la codebase. En l'occurrence, ces derniers jours, le serveur https://colibris.social/ avait du mal à répondre (l'url https://colibris.social/.well-known/void renvoyait une erreur 500), ce qui bloquait le chargement d'Archipelago.

Est-ce qu'on ne devrait pas juste avoir le middleware local par défaut dans ce repo ? Et charge aux instances qui déploient leur Archipelago de rajouter ces serveurs externes à leur convenance ?

Par ailleurs, le middleware local est nommé "av" (pour Assemblée Virtuelle ?), je propose de le renommer en "local" pour ne pas coupler le code fonctionnel du site au nom de l'organisation.

Si cette modification ne correspond pas, j'ai également proposé une solution palliative sur Semapps : https://github.com/assemblee-virtuelle/semapps/pull/1112 pour gérer les erreurs 500.

simonLouvet commented 1 year ago

Je sui d'accord avec cette porposition. des instruction dans le readme ou un renvoie à une docuementation externe seraient utile pour indiquer comment interopérer son archipelago avec d'autres.

srosset81 commented 1 year ago

Cela avait été ajouté car c'était la seule manière de tester certaines fonctions d'interopérabilités.

Je proposerai plutôt de commenter les serveurs cdlt et colibris afin que ça serve d'exemple...

Quand à documenter ces fonctionnalités, il faudrait une documentation spécifique à Archipelago, qui n'existe pour le moment pas.

simonLouvet commented 11 months ago

Cela avait été ajouté car c'était la seule manière de tester certaines fonctions d'interopérabilités.

Je proposerai plutôt de commenter les serveurs cdlt et colibris afin que ça serve d'exemple...

Quand à documenter ces fonctionnalités, il faudrait une documentation spécifique à Archipelago, qui n'existe pour le moment pas.

Pour l'instance de l'AV , je pense qu'il faudrait un repo qui surcharge le fichier dataservers.js spécifique à l'AV avec colibris, CDLT, DP

srosset81 commented 11 months ago

Il suffit de mettre ça là: https://github.com/assemblee-virtuelle/deployment_semapps_av/tree/master/frontend/app

mguihal commented 11 months ago

Suite à cette discussion, j'ai :

Modifié ma PR pour commenter au lieu de supprimer les serveurs externes dans ce fichier, ça permet de servir de documentation sommaire pour les futures instances
Créé la PR suivante sur l'instance de l'Archipelago de l'AV pour rajouter les deux dataServers présents ici : https://github.com/assemblee-virtuelle/deployment_semapps_av/pull/24

Est-ce que ça vous va ?