Piwigo.org

Eric · 2007-06-08 17:44:40

Bonjour à tous.

Je viens de découvrir un gros bug sur ces deux plugins :

Lorsqu'un visiteur s'inscrit, après avoir renseigné tous les champs et cliqué sur le bouton "Register", un message d'erreur apparait :

You are not authorized to access the requested page

Le visiteur est bien inscrit sur la galerie mais pas sur le forum !

Chose étrange, lors de tous mes tests en local cela fonctionnait très bien. Mais j'étais en RC2 à ce moment. Maintenant, en 1.7.0, le bug se reproduit aussi en local. Je ne comprend pas le pourquoi de ce message. Il semble que mon plugin veuille accéder à une partie réservée à l'admin mais je ne vois vraiment pas quoi...

En plus, et cela conforte mon analyse, lorsqu'un admin ajoute lui même un utilisateur via le panneau d'administration \ Utilisateurs, çà fonctionne ! C'est bien qu'il y a quelque chose qui cloche pour un visiteur lambda.

Je vais avoir besoin des lumières des spécialistes php pour remettre à jour ce plugin.

Merci d'avance pour votre aide.

NB : les sources du/des plugin(s) sont dans les extensions. Sinon, je peux poster ici les parties de code qui me semble aller de travers.

Eric · 2007-06-08 19:30:58

J'ai trouvé !

Il n'y a plus de trigger "register_user" dans register.php ! Le trigger n'est dispo que lorsque l'admin ajoute lui-même un utilisateur via le panneau d'admin.

Mais c'est pas normal çà... Si ?

En tout cas, ce n'est pas fait pour arranger mes affaires... Avec çà, les deux plugin Register_Pun/PhpBB sont morts.

Je vais ressortir la RC2 que j'ai utilisé pour le dev et revoir cette histoire de trigger.

rub · 2007-06-08 19:34:18

Le trigger est dans functions_user.inc.php et normalement est appelé quelque soit la méthode utilisée.

Eric · 2007-06-08 19:41:19

Exact... Je me suis emporté car je viens juste de basculer ma galerie en ligne vers la 1.7.0, tout content de pouvoir utiliser mon plugin... qui ne fonctionne plus.

Ceci dit, je pige encore moins le pourquoi de la présence de ce message : "You are not authorized to access the requested page" après l'inscription d'un utilisateur.

rub · 2007-06-08 19:50:50

On dirait que ton user est crée avec un status guest et que par conséquent la redirection vers index.php renvoie une erreur.

Dans indentification.php

Code:

$redirect_to = '';
if ( !empty($_GET['redirect']) )
{
  $redirect_to = urldecode($_GET['redirect']);
  if ( ... )
  {
    array_push($errors, l10n('access_forbiden'));
  }
}

Eric · 2007-06-08 20:57:49

rub a écrit:
On dirait que ton user est crée avec un status guest et que par conséquent la redirection vers index.php renvoie une erreur.

Euh... oui, d'accord... Mais c'est normal çà.

Une personne qui arrive sur une galerie est forcément en status guest jusqu'à ce qu'elle se soit inscrite. Elle passe ensuite en status visiteur.

Mais comment corriger çà dans mon plugin ? Je pensais que le trigger était appelé *après* que le visiteur se soit inscrit. Donc, logiquement, le visiteur n'est plus en guest...

A moins qu'il ne faille définir le status dans le plugin ? J'avoue que je suis un peu paumé, là...

Merci en tous cas pour ton aide.

Eric · 2007-06-08 21:33:07

L'ajout d'un utilisateur est traité dans le fichier reg_punbb_adduser.php qui commence ainsi :

Code:

...
// Needed to check the email var
include_once (PHPWG_ROOT_PATH.'/admin/user_list.php');

// In Admin user_list add user function, the email var is called 'email' and not 'mail_address' as in register.php
// So we have to get the related email var from user_list
$mail = $_POST['email'];

// Load Plugin settings from database
load_conf_from_db('param like \'punbb\\_%\'');

global $conf;
...

Est ce que le fait de faire appel au fichier PWG/admin/user_list.php pourrait expliquer l'interdiction d'accès ?

Eric · 2007-06-08 22:09:18

C'est bon, j'ai trouvé la parade.

En fait, ce n'est pas parce que l'utilisateur qui s'inscrit est en status Guest ou Normal (visiteur) que le pb se produit. Mais bien parce que l'utilisateur n'est pas admin !

Et comme je fais appel à un retour de fonction de l'admin par "include_once (PHPWG_ROOT_PATH.'/admin/user_list.php');", il y a le message d'interdiction.

J'ai corrigé ma fonction adduser pour faire le test de position (admin.php ou register.php) comme ceci :

Code:

...
// Check wich email var is used
if (defined('IN_ADMIN') and IN_ADMIN) /* This is for adding a user in admin panel */
  {
    $mail = $_POST['email'];
...

En enlevant l'appel à admin/user_list.php, je n'ai plus d'erreur et l'inclusion des inscriptions de PWG dans PunBB/PhpBB fonctionne. Je m'empresse de publier une nouvelle version corrigée des deux plugins !

Merci encore à toi, rub, pour m'avoir mis sur la voie. Je pense que j'aurais tourné en rond sur de fausses pistes encore longtemps...

rub · 2007-06-08 23:31:40

Eric a écrit:
Merci encore à toi, rub, pour m'avoir mis sur la voie. Je pense que j'aurais tourné en rond sur de fausses pistes encore longtemps...

De rien mais tu étais parti sur la bonne voie avec le problème du à l"admin ;-)

Nicco · 2008-02-25 00:13:01

voila un poste pour annoncer un nouveau plugin sur le sujet :

le sujet sur le forum = http://forum.phpwebgallery.net/viewtopic.php?id=12780

le plugin dans les extensions = http://phpwebgallery.net/ext/extension_view.php?eid=217

a+

Piwigo.org

Annonce

#1 2007-06-08 17:44:40

[1.7.0 - Plugin Register_PunBB et PhpBB] Gros bug sur regsiter !!

#2 2007-06-08 19:30:58

Re: [1.7.0 - Plugin Register_PunBB et PhpBB] Gros bug sur regsiter !!

#3 2007-06-08 19:34:18

Re: [1.7.0 - Plugin Register_PunBB et PhpBB] Gros bug sur regsiter !!

#4 2007-06-08 19:41:19

Re: [1.7.0 - Plugin Register_PunBB et PhpBB] Gros bug sur regsiter !!

#5 2007-06-08 19:50:50

Re: [1.7.0 - Plugin Register_PunBB et PhpBB] Gros bug sur regsiter !!

Code:

#6 2007-06-08 20:57:49

Re: [1.7.0 - Plugin Register_PunBB et PhpBB] Gros bug sur regsiter !!

rub a écrit:

#7 2007-06-08 21:33:07

Re: [1.7.0 - Plugin Register_PunBB et PhpBB] Gros bug sur regsiter !!

Code:

#8 2007-06-08 22:09:18

Re: [1.7.0 - Plugin Register_PunBB et PhpBB] Gros bug sur regsiter !!

Code:

#9 2007-06-08 23:31:40

Re: [1.7.0 - Plugin Register_PunBB et PhpBB] Gros bug sur regsiter !!

Eric a écrit:

#10 2008-02-25 00:13:01

Re: [1.7.0 - Plugin Register_PunBB et PhpBB] Gros bug sur regsiter !!

Pied de page des forums