Ticket #708 (closed defect: fixed)

Opened 2 years ago

Last modified 2 months ago

Corriger la note sur les caractères autorisés (ie. - n'est pas permis)

Reported by: anonymous Assigned to: anonymous
Priority: high Milestone: alternc-0.9.9
Component: Bureau: membres Version: alternc-0.9.5
Severity: text Keywords:
Cc:

Description

bonjour,

lorsque l'on crée un membre, on saisit son login. Les seuls caractères autorisés pour le login sont a-z 0-9 et -. Or si on met un login du nom : ----- , il prend pas en compte la validité du login.

soit on accepte un login avec -----

soit on dit pourquoi on refuse : "un caractére alphanumérique obligatoire"

c'est pas génant en soi mais une erreur et une erreur !!!

++

Change History

05/29/06 16:38:04 changed by anonymous

  • version changed from 0.9 to 0.9.5.

06/01/06 11:24:37 changed by anonymous

bonjour,

il rejoint celui-ci #427

à voir donc

++

06/05/06 06:20:19 changed by anonymous

  • severity changed from block to text.
  • summary changed from Création de membre : Les seuls caractères autorisés pour le login sont a-z 0-9 et - défectueux to Corriger la note sur les caractères autorisés.

11/19/07 22:46:18 changed by anarcat

  • summary changed from Corriger la note sur les caractères autorisés to Corriger la note sur les caractères autorisés (ie. - n'est pas permis).
  • milestone changed from 1.0 to 0.9.8.

04/11/08 14:17:11 changed by mlutfy

J'ai corrigé ceci dans le hg de Koumbit: 

    // Vérification de la conformité du login + doit commencer par lettre ou chiffre (pas tiret)
    $login=strtolower($login);
    if (!ereg("^[a-z0-9][a-z0-9-]*$",$login)) { //$
      $err->raise("admin",10);
      return false;
    }

i.e. le regexp vérifie d'abord pour [a-z0-9], puis [a-z0-9-]. Notre qu'un login d'une lettre ou chiffre serait accepté. Je peux committer direct dans le trunk?

Je serais tenté à imposer un login qui commence par une lettre, et que le login ait au moins 3 caractères. Ça affecterait uniquement les nouveaux logins, donc pas d'impact sur les installations existantes.

Qu'en pensez-vous?

mathieu (bgm sur freenode)

04/11/08 15:13:22 changed by mlutfy

Petit détail: le fichier à modifier est class/m_admin.php :-)

04/13/08 18:46:28 changed by anarcat

  • priority changed from normal to high.

07/10/08 22:51:05 changed by anarcat

  • status changed from new to closed.
  • resolution set to fixed.

(In [2256]) remove '-' from the list of authorized characters, it's too buggy for production.

Closes: #708