LINCnil / pia-back

Programme développé avec le framework RubyOnRails mettant à disposition une API RESTful à destination des outils PIA et PIA-APP. | Program developped with RubyOnRails providing a RESTful API for the PIA and PIA-APP applications.
GNU General Public License v3.0
156 stars 66 forks source link

Impossible d'activer l'authentification #196

Open Marthym opened 1 year ago

Marthym commented 1 year ago

Bonjour,

Je suis partie de l'installation docker proposé ici (merci à lui). Tout fonctionne bien si ce n'est que je n'ai pas réussi a activer l'authentification. J'ai ajouté dans le dockerfile la variable d'env ENABLE_AUTHENTICATION=true.

Quand j'essaye de jouer les commandes présentent dans la documentation, depuis l'intérieur du container, je tombe sur cette erreur

irb(main):001:0> Doorkeeper::Application.create(name: "PIA", redirect_uri: "urn:ietf:wg:oauth:2.0:oob", scopes: ["read", "write"])
Traceback (most recent call last):
        1: from (irb):1
NameError (uninitialized constant Doorkeeper)

Je ne suis pas un grand habitué de Ruby donc j'ignore comment régler ça ni où chercher ?

Merci pour votre aide.

brunto commented 1 year ago

Bonjour @Marthym,

Le Dockerfile fait référence à une version de pia-back qui ne comprend pas encore l'authentification.

Vous pouvez remplacer la ligne 15 de ce fichier : https://github.com/kosmas58/pia-docker/blob/master/docker/cnil-pia-back/Dockerfile#L15 16 17 et 18

Par :

RUN wget https://github.com/LINCnil/pia-back/archive/refs/heads/master.zip \
 && unzip pia-back-master.zip \
 && rm pia-back-master.zip \
 && mv pia-back-master /var/www

Cela devrait fonctionner.

Marthym commented 1 year ago

Ok merci de votre réponse, je vais faire ça.

Vous pensez tagger une version bientôt. Baser l'installation sur master n'est pas très reproductible ?

kevin-atnos commented 1 year ago

Une nouvelle version / nouveau tag doit sortir très prochainement, elle est en cours de recette / validation par la CNIL. Elle amènera le module d'authentification. Cela sera plus simple d'utiliser cette nouvelle version plutôt que de se baser sur master.