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

Interface Admin : Ajout de champ token dans la table gn_exports.cor_exports_roles et merge de la vue "associer un rôle à un export" dans la vue "Export" #131

Closed
andriacap opened this issue May 10, 2023 · 3 comments

Comments

@andriacap
Copy link
Contributor

No description provided.

@camillemonchicourt
Copy link
Member

Actuellement il faut passer un login / mot de passe pour accéder à l'API json d'un export.
Il est proposé de passer plutôt par un token, plus facile à partager et utiliser quand on interroge une API.
Pour cela, on ajoute un champs token dans la table gn_exports.cor_exports_roles.

@andriacap andriacap changed the title Interface Admin : creer table pour associer token à un id_role Interface Admin : Ajout de champ token dans la table gn_exports.cor_exports_roles et merge de la vue "associer un role à un export" dans la vue "Expor" May 10, 2023
@lpofredc
Copy link
Contributor

Ok, ce n'est pas sans conséquences sur GN2PG. qui devra utiliser les 2 dispositifs d'authentification, le login pour récupérer l'url du module d'export, le token pour récupérer l'export ? A moins que l'on ne puisse utiliser les deux pour l'export (authentification clasdique OU token) ?

@camillemonchicourt
Copy link
Member

camillemonchicourt commented May 10, 2023

Salut, on a pensé à GN2PG qui aurait tout intérêt à passer par un token, à terme.
Mais on a gardé la rétrocompatibilité avec l'authentification à l'API d'un export avec login/mdp.

On avait pas en tête que GN2PG utilisait aussi une autre route pour récupérer l'URL du module Export.
A priori, il nous semble que c'est dommage de devoir générer dynamiquement l'URL de l'API et qu'il vaudrait mieux passer à GN2PG directement l'URL complète de l'API de l'export sans chercher à la générer dynamiquement.
Exemple : https://demo.geonature.fr/geonature/api/exports/api/1
Ainsi GN2PG n'aurait plus besoin d'un login/mdp mais uniquement d'un token.

@camillemonchicourt camillemonchicourt changed the title Interface Admin : Ajout de champ token dans la table gn_exports.cor_exports_roles et merge de la vue "associer un role à un export" dans la vue "Expor" Interface Admin : Ajout de champ token dans la table gn_exports.cor_exports_roles et merge de la vue "associer un rôle à un export" dans la vue "Export" May 10, 2023
@andriacap andriacap moved this from Backlog to In Progress in Refonte module EXPORT May 11, 2023
@camillemonchicourt camillemonchicourt moved this from In Progress to Done in Refonte module EXPORT May 14, 2023
@camillemonchicourt camillemonchicourt moved this from Done to In Progress in Refonte module EXPORT May 14, 2023
@camillemonchicourt camillemonchicourt moved this from In Progress to Done in Refonte module EXPORT May 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

4 participants