Ticket #427 (closed defect: wontfix)

Opened 8 years ago

Last modified 7 months ago

Autoriser le tiret dans les noms d'usagers

Reported by: arnaud_lb Owned by: anonymous
Priority: normal Milestone: alternc-1.999
Component: Bureau: membres Version: alternc-0.9.3
Severity: feature Keywords:
Cc:

Description (last modified by fufroma) (diff)

Les seuls caractères autorisés pour le login sont a-z 0-9 et -

Dans m_admin::add_mem() on a ça:

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

Donc le tiret n'est pas autorisé.

Il faudrais remplacer l'expression par: "^[a-z0-9-]*$"

Change History

comment:1 Changed 8 years ago by arnaud_lb

Sauf que en fait ça cause des problèmes avec les base de données. Il faut protéger les nom de tables avec des . Il y a juste quelques lignes à modifier dans bureau/class/m_mysql.php:

  • $db->query('SHOW TABLE STATUS FROM '.$c[$key]db?.';');

+ $db->query('SHOW TABLE STATUS FROM '.$c[$key]['db'].';');

  • $db->query("CREATE DATABASE $dbname;");

+ $db->query("CREATE DATABASE $dbname;");

  • $db->query("DROP DATABASE $dbname;");

+ $db->query("DROP DATABASE $dbname;");

  • $db->query("CREATE DATABASE ".$mem->userlogin?.";");

+ $db->query("CREATE DATABASE ".$mem->user["login"].";");

comment:2 Changed 8 years ago by anarcat

  • Summary changed from Les seuls caractères autorisés pour le login sont a-z 0-9 et - to Autoriser le tiret dans les noms d'usagers
  • Severity changed from minor to feature
  • Milestone set to 0.9.3

comment:3 Changed 8 years ago by anarcat

  • Summary changed from Les seuls caractères autorisés pour le login sont a-z 0-9 et - to Autoriser le tiret dans les noms d'usagers

comment:4 Changed 7 years ago by anarcat

  • Type set to defect
  • Milestone changed from 0.9.3 to 2.0

comment:5 Changed 7 years ago by anarcat

voir aussi #783.

comment:6 Changed 6 years ago by anarcat

(In [1745]) use same regexp for allowed usernames as in the php code (m_admin::add_mem())

do not quote_meta in shell command, since all variables are now considered safe

see #998 see #427

comment:7 Changed 6 years ago by anarcat

  • Description modified (diff)
  • Milestone changed from 2.0 to 1.0

comment:8 Changed 5 years ago by anarcat

  • Milestone changed from 1.0 to 0.9.8

comment:9 Changed 5 years ago by anarcat

  • Milestone changed from 0.9.8 to 1.0

On va commencer par corriger la note, puis après on changera la fonctionalité.

comment:10 Changed 7 months ago by fufroma

  • Status changed from new to closed
  • Resolution set to wontfix
  • Description modified (diff)
Note: See TracTickets for help on using tickets.