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

Login non transfé #246

Closed
Trickscenique opened this issue Feb 27, 2023 · 8 comments
Closed

Login non transfé #246

Trickscenique opened this issue Feb 27, 2023 · 8 comments

Comments

@Trickscenique
Copy link

Trickscenique commented Feb 27, 2023

Décrivez le bogue
Lors de la connexion le login n'est pas enregistré en session ce qui ne permet pas d'accéder à l'administration.

Infos GRR
Version de GRR : 4.0.2
Version PHP : Php 8.1

Solution
Correctif pour le fichier include/session.inc.php Ligne 614
$_SESSION['login'] = /*$row['LOGIN'];*/ $_login;

Cela vient de la fonction grr_sql_row_keyed() qui semble supprimer le champs indexé du tableau de valeur.

@ynaessens
Copy link
Collaborator

ynaessens commented Mar 1, 2023

Bonjour,
à mon avis, le problème ne vient pas de grr_sql_row_keyed() mais du fait que LOGIN est distinct de login.
Dans le script vous devriez avoir $_SESSION['login'] = $row['login'];
Cordialement,
YN

@Trickscenique
Copy link
Author

Je vais essayer comme ceci, cependant en base de donnée le champs s'appelle bien LOGIN (en majuscule).

@ynaessens
Copy link
Collaborator

Dans quelle table ?

@Trickscenique
Copy link
Author

Autant pour moi, c'est dans la table grr_log, il me semblait l'avoir vu dans la table grr_utilisateurs.
Par contre sur mon installation, ma modification était bien nécessaire pour que l'authentification reste en $_SESSION.
Je n'ai pas eus le temps de vérifier avec la clef login en minuscule.

@ynaessens
Copy link
Collaborator

Pas de souci.
Je préfère l'appel par $row puisqu'il y a nettoyage de $_login lors de la requête.

@JeromeDevome
Copy link
Owner

Bonjour @Trickscenique ,
J'ai corrigé l'état de l'utilisateur dans la version 4.0.2 qui empêché la visualisation du lien admin. Et vu la date du post vous n'êtes pas en 4.0.2 mais plutôt une version RC.

@JeromeDevome
Copy link
Owner

Bonjour @Trickscenique , votre problème est-il résolut avec la dernière version ?

@Trickscenique
Copy link
Author

Trickscenique commented Mar 21, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants