dbarzin / mercator

Mapping the information system / Cartographie du système d'information
GNU General Public License v3.0
247 stars 39 forks source link

Intégration de Keycloak et Ajustements API Mercator #752

Closed BadaouiMu closed 3 months ago

BadaouiMu commented 3 months ago

Salut Didier,

Récemment, j'ai bossé sur l'intégration de Keycloak sur des applications PHP, en utilisant Mercator comme base. J'ai fait quelques modifs et ça marche.

Un autre point que je voulais te mentionner (pas urgent pour moi pour le moment et je sais pas si c'est déjà réglé dans tes dernières versions, je n'ai pas eu le temps de tester) :

J'ai fait ces modifs dans ce Pull.

Une dernière demande/question : je n'arrive jamais à faire fonctionner les API après avoir fini l'installation de Mercator. Du coup, pour que les API marchent pour moi, je dois suivre cet acheminement :

cp .env.example .env
sudo php artisan migrate --seed
sudo php artisan passport:install
sudo php artisan passport:keys
sudo php artisan key:generate
sudo php artisan config:clear
mysql mercator < mercator_data.sql
sudo gzip -d mercator_cpe.sql.gz
sudo mysql mercator < mercator_cpe.sql
sudo php artisan serve --host 0.0.0.0 --port 8000

Si je fais autre chose, ça ne marche pas. Je ne sais pas si c'est seulement pour moi ou pas.

Merci d'avance, Mustapha

dbarzin commented 3 months ago

Merci, très bonne contribution !

Toutes les commandes que tu cites pour fait fonctionner l'API se trouvent dans la procédure d'installation. Peut-être faut-il revoir la procédure d'installation ?

dbarzin commented 3 months ago

Peux-tu mettre à jour la procédure d'installation concernant la configuration de Keycloak ?

BadaouiMu commented 3 months ago

En effet, il n'y a pas vraiment beaucoup de choses à faire. Il suffit de modifier le fichier .env et de le remplir avec les informations sur Keycloak. Cependant, il peut y avoir des configurations à faire sur Keycloak. Si tu veux, je peux essayer d'écrire une procédure détaillée sur ce sujet cette semaine

Par ailleurs, pourrais-tu vérifier le petit truc sur les API ? Je suis censé écrire une documentation sur la mise à jour et l'activation des API de Mercator, mais je n'arrive pas à le faire fonctionner

dbarzin commented 3 months ago

Tu peux faire directement référence à la documentation de Keycloack en mettant un lien.

dbarzin commented 3 months ago

Je vais regarder pour l'API, mais ce ne sera pas avant ce week-end.

BadaouiMu commented 3 months ago

J'ai modifié les fichiers d'installation pour ajouter la procédure de configuration de Keycloak, et j'ai également ajusté le code afin que le bouton Keycloak n'apparaisse que si le paramètre KEYCLOAK est activé.

BadaouiMu commented 3 months ago

J'ai également modifié la valeur maximale des registres des traitements à 255. Je pense que les DPO ne respectent absolument pas la règle des moins de 32 caractères et donc, s'ils veulent passer à Mercator, cela va être compliqué.

dbarzin commented 3 months ago

L'API fonctionne après avoir suivi la procédure d'installation. Il ne faut pas oublier d'installer Passeport. Le changement relatif à la valeur maximale du nom des registres des traitements ne devrait pas faire partie de cette PR.