[Sécurité] Probleme d'actualisation dans un caddie

**budiste** · 12/12/2005, 09h29

Bonjour,

donc voilà le lien me permettant d'augmenter la quantité d'un produit de un :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
<a href="index2.php?panier=1&action=ajoutun&idpdt=<? echo $cat['IDARTICLE'];?>">
<img src="images/boutonplus.gif" border="0">
</a>

ensuite je vérifies l'action a effectué et elle s'effectue donc voilà le code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
<?
session_start();
if(array_key_exists('action', $_GET)){
	$actionpanier=$_GET['action'];
	$idartsuppr=$_GET['idsuppr'];
	$idproduit=$_GET['idpdt'];
	if ($actionpanier=='supprligne'){
		unset($_SESSION['achats'][$idartsuppr]);
	}
	if ($actionpanier=='vider'){
    	unset($_SESSION['achats']);
	}
	if ($actionpanier=='ajoutun'){
		$nouvelleqte = $_SESSION['achats'][$idproduit]['qte'] + 1;
		$_SESSION['achats'][$idproduit] += $nouvelleqte;
	}
	if ($actionpanier=='supprun'){
		$nouvelleqte = $_SESSION['achats'][$idproduit]['qte'] - 1;
		$_SESSION['achats'][$idproduit] += $nouvelleqte;
	}
}
?>

mon problème est que quand je fais actualiser il recharge la page et augmente donc la page a nouveau de un j'aimerais donc savoir comment faire pour remettre juste le lien d'affichage du caddie, je ne peux pas mettre un headerlocation car il y du html avant le php.

**Mr N.** · 12/12/2005, 09h37

tu peux ajouter un mécanisme de jeton.
Lors de l'affichage d'une page tu génères un jeton que tu injecte dans tout tes liens qui sont critiques (là ou le reload est pénalisant) + tu le stocke en session.
Lorsque tu recois une requete, tu récupères le jeton de la requete et tu le compare avec celui en session. si c'est le meme tu executes la requete, puis tu invalide le jeton en session, et si c'est pas le meme c'est qu'il y a eu un refresh ou autre siouxerie qui mettrait le bazar.
pour ton jeton => rand, uniqid, ...

**budiste** · 12/12/2005, 15h54

1 - J'ai ma page index2.php, qui contient :
* le menu
* une colonne news
* un bas de page
* le corps de la page

2 - Dans le corps de la page, j'ai un include qui me permet t'appeler mais différentes pages, mais ce sont des bouts de tableaux (probleme du a la présentation imposée)

3 - j'ai donc mis dans toutes mes parties du corps de page la ligne suivante comme tu me l'as dis, sauf dans la page du panier :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if(!isset($_SESSION['verif_value'])) $_SESSION['verif_value'] = rand(1,1000000);

4 - dans la page panier voilà mon code maintenant :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
if(array_key_exists('action', $_GET)){
    $actionpanier=$_GET['action'];
    $idartsuppr=$_GET['idsuppr'];
    $idproduit=$_GET['idpdt'];
    if ($actionpanier=='supprligne'){
        unset($_SESSION['achats'][$idartsuppr]);
    }
    if ($actionpanier=='vider'){
        unset($_SESSION['achats']);
    }
    if($_SESSION['verif_value'] == $_GET['verif_value']){
        if ($actionpanier=='ajoutun'){
            $nouvelleqte = $_SESSION['achats'][$idproduit]['qte'] + 1;
            $_SESSION['achats'][$idproduit] += $nouvelleqte;
        }
        if ($actionpanier=='supprun'){
            $nouvelleqte = $_SESSION['achats'][$idproduit]['qte'] - 1;
            $_SESSION['achats'][$idproduit] += $nouvelleqte;
        }
    }
}

5 - j'ai ensuite réouvert le site, recréé un panier, mais quand j'augmente le nombre et que j'actualise j'ai le meme probleme, ca réaugmente

**Mr N.** · 12/12/2005, 16h01

Tu as oublié une étape : invalider le jeton en session !

**budiste** · 12/12/2005, 16h06

heu...

:

**Mr N.** · 12/12/2005, 16h12

Ben oui. Le jeton est à usage unique. donc une fois qu'il est utilisé (formulaire traité), tu le supprimes de la session (unset)

**budiste** · 12/12/2005, 16h24

je le supprime apres mes if pour ajouter un et supprimer un ?

**budiste** · 12/12/2005, 16h35

J'ai fais comme ça mais ça ne change toujours rien

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
<?
session_start();
if(array_key_exists('action', $_GET)){
	$actionpanier=$_GET['action'];
	$idartsuppr=$_GET['idsuppr'];
	$idproduit=$_GET['idpdt'];
	if ($actionpanier=='supprligne'){
		unset($_SESSION['achats'][$idartsuppr]);
	}
	if ($actionpanier=='vider'){
    	unset($_SESSION['achats']);
	}
	if($_SESSION['verif_value'] == $_GET['verif_value']){
    	if ($actionpanier=='ajoutun'){
			$nouvelleqte = $_SESSION['achats'][$idproduit]['qte'] + 1;
			$_SESSION['achats'][$idproduit] += $nouvelleqte;
		}
		if ($actionpanier=='supprun'){
			$nouvelleqte = $_SESSION['achats'][$idproduit]['qte'] - 1;
			$_SESSION['achats'][$idproduit] += $nouvelleqte;
		}
		unset($_SESSION['verif_value']);
	}
}
?>

**Mr N.** · 12/12/2005, 16h55

Cadeau :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
<?php
 
session_save_path(dirname(__FILE__));
session_start();
 
$msg = '';
if (isset($_REQUEST['action']) && isset($_REQUEST['jeton'])) {
    if (isset($_SESSION['jeton']) && $_SESSION['jeton'] == $_REQUEST['jeton']) {
        $msg = 'action effectuée !!!';
        unset($_SESSION['jeton']);
    } else {
        $msg = '/!\ action pas effectuée /!\\';
    }
}
$_SESSION['jeton'] = rand(1, 10000);
 
echo $msg;
?>
<form action="" method="POST">
<fieldset>
    <legend>Formulaire</legend>
    <input type="hidden" name="jeton"  value="<?=$_SESSION['jeton']?>" />
    <input type="submit" name="action" value="Action !" />
</fieldset>
</form>
<a href="?jeton=<?=urlencode($_SESSION['jeton'])?>&amp;action=lien">Une autre action par un lien</a>

**budiste** · 12/12/2005, 17h04

la position de mon unset ne reviens pas au meme

**budiste** · 13/12/2005, 09h54

j'ai pas mis unset je regénère le jeton et ca marche, merci beaucoup pour ton aide :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
if(array_key_exists('action', $_GET)){
	$actionpanier=$_GET['action'];
	$idartsuppr=$_GET['idsuppr'];
	$idproduit=$_GET['idpdt'];
	if ($actionpanier=='supprligne'){
		unset($_SESSION['achats'][$idartsuppr]);
	}
	if ($actionpanier=='vider'){
    	unset($_SESSION['achats']);
	}
	if($_SESSION['verif_value'] == $_GET['verif_value']){
    	if ($actionpanier=='ajoutun'){
			$nouvelleqte = $_SESSION['achats'][$idproduit]['qte'] + 1;
			$_SESSION['achats'][$idproduit] += $nouvelleqte;
		}
		if ($actionpanier=='supprun'){
			$nouvelleqte = $_SESSION['achats'][$idproduit]['qte'] - 1;
			$_SESSION['achats'][$idproduit] += $nouvelleqte;
		}
		$_SESSION['verif_value'] = rand(1,1000000);
	}
}

[Sécurité] Probleme d'actualisation dans un caddie

Langage PHP

Discussions similaires

Partager

Partager