Reprise du message précédent :

heureusement que c est pas stiffler qui poste j aurais été decu

c'est vrai !
bon maintenant, Florentg ta chambre !

Quelqu'un pourrait-il me dire pourquoi ceci ne m'envoie qu'un mail sur adresse1@mail.com et pas sur les autres ?
 

$mails = array('adresse2@mail.com', 'adresse3@mail.com');
$headers = 'Bcc: ' . implode(',', $mails) . "\r\n";
mail('adresse1@mail.com', 'Sujet', 'Message', $headers);

ptet parce que t'as mis cette adresse dans l'argument réservé au destinataire ?

 
Gné ?
 
Normalement c'est censé envoyer un mail sur adresse1, et une copie cachée sur adresse2 et adresse3 là non ?

j'en sais rien moi, tsé, moi et le php hein

$mails = array('adresse2@mail.com', 'adresse3@mail.com');
$headers = 'Bcc: ' . implode(',', $mails) . "\r\n";
mail('adresse1@mail.com', 'Sujet', 'Message', $headers);

Use fuckin' Google  
 

 
Use fuckin' cerveau
 
1> Le serveur est sous Linux  
2> Si je veux envoyer un message sous forme de copie cachée, c'est peut-être qu'il y a une raison non ?

Déjà faut pas utiliser la fonction mail() hein
Y a des libs spécialisées dans les envois de mail qui font bcp mieux (j'ai utilisé Swift récemment pour un mailing, bien sympa).

Et donc
 
Si t'avais lu le message en entier t'aurais vu que ça change rien

Ben grâce à lui tu sais ptet que tu peux pas

$sql = "SELECT pseudo, mdp, admin FROM ".TBL_NEWS_CAT." where pseudo = '".$POST_['pseudo']."' AND mdp = MD5('".$_POST['mdp']."') AND admin = '1'";
    $sql = mysql_query($sql);
    if($sql)
    {
        $_SESSION['auth'] = true;
        header('Location: admin.php');
    }

 
Pour en revenir à ça, j'ai ce bout de code que j'inclus sur toutes mes pages admin :
 

<?php
session_start();
if($_SESSION['auth'] != true)
{
    echo "Pas auth.";
    exit();
}
?>

 
Le truc c'est que lorsque je me logue, j'ai directement le droit au message "Pas auth", alors que je suis bien logué

 
Nan mais laule quoi  
 
'fin bref, en fait suffisait de mettre "bcc" tout en minuscule, et ça passe ...
 
(pis franchement à vous lire, on se demande pourquoi ils auraient foutu cette possibilité ...)

<?php
session_start();
if($_SESSION['auth'] != true)
{
    echo "Pas auth.";
    exit();
}
?>

 
Tu as bien un session_start() pour la 1ère partie ?

<?php
session_start();
if($_SESSION['auth'] != true)
{
    echo "Pas auth.";
    exit();
}
?>

On t'as déja dit que ton code c'était une merde pas sécurisée ou je débarque?

J'ai modifié la requête, mais de toute façon je ne l'avais pas encore sécurisé car là n'était pas la question

SELECT pseudo, mdp, admin FROM ".TBL_MEMBRES." where pseudo = '".mysql_real_escape_string($_POST['pseudo'])."' AND mdp = MD5('".mysql_real_escape_string($_POST['mdp'])."') AND admin = '1'

Edit : j'avais effectivement oublié le session_start() sur ma page de login.

Laisse-moi juste réussir un hello word sous rosasm, et promis je vais dormir

 
\o/

 
Merci bien

$sql = "SELECT pseudo, mdp, admin FROM ".TBL_NEWS_CAT." where pseudo = '".$POST_['pseudo']."' AND mdp = MD5('".$_POST['mdp']."') AND admin = '1'";
    $sql = mysql_query($sql);
    if($sql)
    {
        $_SESSION['auth'] = true;
        header('Location: admin.php');
    }

 
Salut,
 
je debarque pe mais tu traites jamais le resultat de ta requete ?  
Tu verifies juste qu'il y a pas d'erreur de syntaxe ?
Il me semble que mysql_query renvoie false uniquement en cas d'erreur (syntaxe ou droits sql), sinon la fonction renvoie une ressource à traiter, mais je peux me tromper.....
car sans fetch ou autre, la meme si je rentre 'jean-pierre' en pseudo avec 'dernier mot?' en mdp je suis loggé.... (enfin je pense)

Io, il y a un moyen d'utiliser : opacity: 0.9 en CSS est rester compliant ?
 
Jai tester de mettre ça dans un JS, car jai besoin de ce hack que qd JS est activé...Mais ça le fait pas trop beau... sur un unload
 
Sinon reste la soluce le mettre dans le code direct...
 
EDIT:
 
Op mis ça:
 

 
Mais si ia mieux

 
Euh non, ça fonctionne très bien comme ça.

M'étonnerait ça
Tu dois pouvoir te logguer avec n'importe quel login/mdp avec ton système a priori. Vu que la requête est correctement exécutée, mysql_query renvoie une valeur de type resource qui est évaluée à true dans ton if, quelque soit le nombre de ligne renvoyé par la requête.

 
C'est exact  
 
Moi qui pensait pouvoir tout simplifier via ma requête, ce que j'ai fait n'a servit à rien du tout

Questionne conne :
normalement, la requête juste en dessous est sensé me donner un résultat ou aucun.
Je fais comment pour savoir si elle me donne un résultat ?
 
SELECT pseudo, mdp, admin FROM ".TBL_MEMBRES." where pseudo = '".mysql_real_escape_string($_POST['pseudo'])."' AND mdp = MD5('".mysql_real_escape_string($_POST['mdp'])."') AND admin = '1'
 
J'ai honte là

mysql_num_rows() ou alors vérifier que le mysql_fetch* renvoie pas false

Ouais, ok, obligé de faire comme ça.

Vous pouvez me dire ce que vous pensez de cette page de login s'il vous plait ?
 
Ce qui ne va pas et comment je pourrai l'améliorer ?
 
Merci.
 

<?php
session_start();
 
if($_SESSION['auth'] == true)
{
    header('Location: admin.php');
}
 
$_SESSION['auth'] = false;
 
if($_POST)
{
    require(dirname(__FILE__)."/includes/config.inc.php" );
    require(dirname(__FILE__)."/includes/connect.inc.php" );
    
    $sql = "SELECT COUNT(pseudo) AS nb FROM ".TBL_MEMBRES." where pseudo = '".mysql_real_escape_string($_POST['pseudo'])."' AND mdp = MD5('".mysql_real_escape_string($_POST['mdp'])."') AND admin = '1'";
    $res = mysql_query($sql);
    
    $nbr = mysql_fetch_assoc($res);
    
    if($nbr['nba'] == 1)
    {
        $_SESSION['auth'] = true;
        $_SESSION['pseudo'] = $_POST['pseudo'];
        header('Location: admin.php');
    }    
}    
?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
    <title>Connexion à l'espace d'administration</title>
    <link href="styles/defaut.css" type="text/css" rel="stylesheet" />
</head>  
<body>
 
<h1>Connexion à l'espace d'administration</h1>
 
<form action="index.php" method="post">
<p><label>Pseudo :</label>
<input type="text" name="pseudo" /></p>
 
<p><label>Mot de passe :</label>
<input type="password" name="mdp" /></p>
 
<p><input type="submit" value="Se connecter" /></p>
</form>
 
</body>
</html>

...

 
Perso, ce que je chengerais :

• après ta requête SQL, fais le test sur le cle "nb" au lieu de "nba" ...
• fais gaffe à ce que tu stockes dans ta session. Là, tu y mets le pseudo utilisateur tel quel. Si jamais plus loin, tu te dis "ok, ça vient de ma session, j'ai rien à craindre" et que t'oublies de faire un nettoyage dessus, blam.

Le nba était une faute de frappe, j'avais la page ouverte et j'ai touché mon clavier, mais ça fonctionne très bien
 
Par contre je comprend pas trop pour le second point.

Perso, je ferais :
$sql = "SELECT 1 FROM ".TBL_MEMBRES." where pseudo = '".mysql_real_escape_string($_POST['pseudo'])."' AND mdp = MD5('".mysql_real_escape_string($_POST['mdp'])."') AND admin = '1'";
$res = mysql_query($sql);
list($exist)= mysql_fetch_row($sql);
 
if(!empty($exist))
 
 
Après, niveau perf, c'est mieux mais c'est de la branlette de mouche, c'est + les gouts et les couleurs.
 
 

Du code...

Pense à mettre des exit après les header('Location...');
C'est pas parce qu'il y a un header('Location') que le code qui suit ne va pas être exécuté

 
SELECT 1
list($exist)
 

 
D'accord

Si tu fais un "SELECT 1 WHERE ...", et que ta condition est remplie (et donc qu'il faut authentifier l'user), ça te retourne donc la valeur 1, dans ta ressource MySQL.

Quant à :

list($exist) = mysql_fetch_row($sql);

msql_fetch_row va retourner un array, avec une seule valeur.

Le fait d'utiliser list($exist) affecte la première valeur retournée à la variable $exists.

En gros, tu as un array de cette forme :

$mon_array = array('pomme', 'rouge', 'grand');

tu peux faire ça :

list($fruit, $couleur, $taille) = $mon_array;

Je savais pas du tout, merci à vous deux
 
Enfin dans ce cas ça relève plus du chipotage non ?

Oui et non, le SELECT COUNT(pseudo) ne sert à rien dans le cas présent, donc tu peux mettre SELECT 1

Pour le "list" c'est du chipotage oui (je n'ai d'ailleurs jamais utilise list pour faire ça)

Edit : Moi ça serait plutôt

$valid_user = $DB->getVar('SELECT 1 FROM ma_table WHERE ...');

<?php
if($_POST)
{
    require(dirname(__FILE__)."/includes/config.inc.php" );
    require(dirname(__FILE__)."/includes/connect.inc.php" );
 
    $sql = "SELECT COUNT(pseudo) AS nb FROM ".TBL_MEMBRES." where pseudo = '" 
                . mysql_real_escape_string($_POST['pseudo']) 
               . "' AND mdp = MD5('" . mysql_real_escape_string($_POST['mdp']) . 
               "') AND admin = '1'";
    $res = mysql_query($sql);
 
    $nbr = mysql_fetch_assoc($res);
 
    if($nbr['nba'] == 1)
    {
        $_SESSION['auth'] = true;
        $_SESSION['pseudo'] = $_POST['pseudo'];
        header('Location: admin.php');
    }
}
?>

Pour ce petit bout de code, je te conseillerais plusieurs trucs.

1) Fait-toi une classe pour les requêtes sql liés à l'authentification.

<?php
class KKZ_Mysql_Auth
{
 
    public function __construct()
    {
    }
 
 
    public function userExists($user, $pass)
    {
        $user = mysql_real_escape_string($user);
        $pass = hash('md5', $pass);
 
        $result = mysql_query(
            'SELECT COUNT(`pseudo`) AS nb FROM ' . TBL_MEMBRES
          . ' WHERE `pseudo`=\'' . $user . '\' AND `pass`=\'' . $pass . '\' AND ADMIN=1');
 
        $row = mysql_fetch_assoc($result);
 
        return (int)$row['nb'] === 1;
    }
 
}
?>

Vaut mieux se faire des classes comme ça avec les requêtes, ça soulagera
grandement ton code au-dessus.

2) Même chose pour la session. Si jamais sur ton site t'as un formulaire
d'inscription, t'aura à 2 endroits le même code pour logger un utilisateur. Si
jamais tu changes la façon de logger (genre en rajoutant des champs), ça fera
duplication de code :

<?php
 
class KKZ_Session
{
 
    private $started;
 
 
    public function __construct()
    {
    }
 
 
    public function logUser($user)
    {
        $this->start();
 
        $_SESSION['auth'] = true;
        $_SESSION['pseudo'] = $user;
    }
 
 
    public function start()
    {
        if(!isset($_SESSION)) {
            session_start();
        }
    }
 
}
?>

Lorsque tu te metteras aux tests unitaires, tu pourra faire un mock de la
classe session, et substituer les méthodes pour tester juste ce qu'elles
reçoivent. Je vais faire un p'tit post dans mon topic pour expliquer tout ça.

Ne fait pas un session_start() à chaque fois, si tu peux t'en passer.
J'ai l'habitude de tester si un cookie avec l'identifiant de session existe,
et si seulement il existe, on démarre la session.

3) Pour la redirection, utilise une adresse absolue, un status code 303, et
fait un exit, sinon le reste est quand-même exécuté :

header('http://' . $_SERVER['SERVER_NAME'] . '/admin.php', true, 303);
exit();

Fait-toi une classe KKZ_HttpResponse avec une belle méthode redirect() qui peut
te gérer ça tout seul

Le contenu de ce message a été effacé par son auteur

Ca devrait aller mieux comme ça :

function selectAllOptions(idSelect) {
    var options = document.getElementById(idSelect).options;
    for (var i = 0 ; i < options.length ; i++) {
        options[i].selected = true;
    }
}

 
Et je te conseille le onsubmit du form plutot que le onclick

Le contenu de ce message a été effacé par son auteur

J'ai énormément de mal à saisir l'intérêt de la chose ... À quoi sert ce SELECT s'il faut de toute façon qu'ils soient tous sélectionnés ?

cytrouille > L'attribut "selected" sert à désigner les options sélectionné par défaut au chargement de la page. Ca ne sert pas à indiquer ou trouver les éléments actuellement sélectionné.
Voilà pourquoi ça ne servait à rien de modifier l'attribut "selected"