[PHP]Używanie GET w wylogowywaniu

[Mari]

Czy używanie metody GET w wylogowywaniu jest bezpieczne?
Podam przykład:

Kod: Zaznacz cały

if($_GET['wyloguj']=='tak')
	{
	echo "Zostałeś wylogowany z serwisu";
	session_destroy();
	}

Ten kod powoduje,że jeżeli wpiszemy w adres przeglądarki www.naszastrona.pl?wyloguj=tak to wyświetli napis "Zostałeś wylogowany z serwisu" i zniszczy sesje ale czy na pewno to jest bezpieczne bo w sumie nie wiem?

[Radzio]

Bez różnicy.

[Mari]

Żeby nie tworzyć nowych tematów opiszę jeszcze jeden problem z formularzem.
Otóż chce aby formularz pokazywał się tylko raz na początku i wtedy gdy wpiszemy złe hasło a na razie pokazuje się dwa razy czyli i ten początkowy i tamten gdy wpiszemy źle hasło.Jak zrobić żeby tylko raz się pokazywało gdy wpiszemy niewłaściwe hasło.
Podaje kod:

Kod: Zaznacz cały

<form action="panel.php" method="post">
	<input type="password" name="pass1" />
	<input type="submit" value="Zaloguj"/>
</form>
	<?php 
	if($_GET['wyl1']=='yes')
	{
	echo "Zostałeś wylogowany z serwisu";
	session_destroy();
	}
	
	if (isset($_POST['pass1']))
	
{
if ($_POST['pass1'] == 'siema')
{
echo 'Jesteś zalogowany';
?>
</br></br></br><a href='panel.php?wyl1=yes'>Wyloguj się</a>
<?php
}
else
{
echo 'Nie jesteś zalogowany';
?>
<form action="panel.php" method="post">
	<input type="password" name="pass1" />
	<input type="submit" value="Zaloguj"/>
</form>
<?php
}
}
?>

Dobra sprawa rozwiązana po prostu dać

Kod: Zaznacz cały

header('location: panel.php');

jeżeli ktoś jest niezalogowany i powracasz tym kodem do pierwotnego formularza.
Jest tylko ten problem,że aby użyć tego to nie możesz nic wysłać przed tym do przeglądarki żadnego tekstu.Czyli kod musimy dać na samym początku.
Pozdrawiam

[pomek]

Czy używanie metody GET w wylogowywaniu jest bezpieczne?

W postaci niezabezpieczonej - nie

A odnosnie formularza, nie lepiej ogarnac to w funkcje? Osobiscie jestem zwolennikiem klas, ale pstryknalbym to w taki sposob:

Kod: Zaznacz cały

function form($error = "") { if(!empty($error)) { echo $error.'<br />'; } 
// kod formularza }

W miejscu gdzie ma sie pojawic formularz pstrzykasz echo $form();
Jesli chcesz wyswietlic blad, to jako wpisujesz go jako parametr funkcji Form.

[Sebastian]

Odkop, ale cóż

if($_GET['wyl1']=='yes')


a co jak ktoś ci nie poda parametru wyl1? :>
if(isset($_GET['wyl1']) && $_GET['wyl1'] == 'yes')

[Bren]