[php] ukrywanie danych w linku Opcje

[pijanyadmin]

obecnie rekord z bazy jest zamieniany na link i w przegladarce wygląda to tak:

Kod

panel.php?gog=poczta&wiad=odczyt&antylopa&id=52

czyli, autor wiadomość "antylopa" id wiadomość w bazie to "52"

odczyt wiadomość wygląda tak:

[PHP] pobierz, plaintext 
<?php
session_start();
require("connect.php");
 
$login = $_SESSION['id'];
 
$quer = mysql_query("SELECT * FROM `wiadomosci` WHERE `do_kogo` = '$login' ORDER BY `id` DESC");
?>
 
 
<?php
while ($wiersz = mysql_fetch_array($quer)) 
{
$i++;
$class = ($i%2==1) ? "red" : "czarny";
echo '<div class="'. $class .'";>';
echo '<div class="pok"><a href="panel.php?gog=poczta&wiad=odczyt&'.$wiersz[od_kogo].'&id='.$wiersz[id].'">'.$wiersz[od_kogo].'</a></div>';
echo '<div class="pok2"> | <a href="npanel.php?gog=poczta&pocz=kasuj&id='.$wiersz[id].'">usuń</a></div>';
echo '<div class="pok2"> | '.$wiersz[time].'</div>';
echo '<div class="pok2">'.$wiersz[temat].'</div>';
echo '</div>';
}
 
?>
[PHP] pobierz, plaintext 

po kliknieciu w link który jest już odpowiednio zamieniony następuje przekierowanie do odczytu danej wiadomość

[PHP] pobierz, plaintext 
<?php
session_start();
require("connect.php");
 
$q = mysql_fetch_array(mysql_query("SELECT * FROM `wiadomosci` WHERE `id` = ".$_GET['id']." LIMIT 0,1"));
 
 
echo 'Od: <b>'.$q[od_kogo].'</b> Nadana:<b>'.$q[time].'</b><br />';
echo 'Temat: <b>'.$q[temat].'</b><br />';
echo ''.$q[wiadomosc].'';
echo '<a href="panel.php?gog=poczta&wiad=odpowiedz'.$_SESSION[od_kogo].'"><br />Odpowiedz NA WIADOMOSC</a>';
 
 
?>
[PHP] pobierz, plaintext 

tylko teraz jak to zabezpieczyć odpowiednio aby w adresie przegladarki podczas odczytywania wiadomosc nie było ani informacji o nadawcy ani o ID wiadomosci. Próbowałem to dać "od_kogo" do sesji... ale jak dodać do niej ID wiadomość? czy są jakieś inne sposoby na to?

Ten post edytował pijanyadmin 3.02.2008, 02:42:02

[kurt]

A nie możesz po prostu zarejestrować zmiennej sesyjnej $id_wiad ?
Rozumiem, że podany przez Ciebie listing dotyczy dwóch plików?

[nowotny]

Możesz sobie zrobić tak:

[PHP] pobierz, plaintext 
<?php
while ($wiersz = mysql_fetch_array($quer)) 
{
$i++;
$class = ($i%2==1) ? "red" : "czarny";
echo '<div class="'. $class .'";>';
 
unset($_SESSION['wiersze']);
$_SESSION['wiersze'][$i]['id']=$wiersz['id'];
$_SESSION['wiersze'][$i]['od_kogo']=$wiersz['od_kogo'];
 
echo '<div class="pok"><a href="panel.php?gog=poczta&wiad=odczyt&num='.$i.'">'.$wiersz[od_kogo].'</a></div>';
echo '<div class="pok2"> | <a href="npanel.php?gog=poczta&pocz=kasuj&num='.$i.'">usuń</a></div>';
echo '<div class="pok2"> | '.$wiersz[time].'</div>';
echo '<div class="pok2">'.$wiersz[temat].'</div>';
echo '</div>';
}
?>
[PHP] pobierz, plaintext 

Czyli do sesji dodajemy tablicę ze spisem wierszy a w URLu przekazujemy indeks w tej tablicy a nie id...

Potem w drugim skrypcie:

[PHP] pobierz, plaintext 
<?php
session_start();
require("connect.php");
 
$num=$_GET['num'];
$id=$_SESSION['wiersze'][$num]['id'];
$od_kogo=$_SESSION['wiersze'][$num]['od_kogo'];
 
$q = mysql_fetch_array(mysql_query("SELECT * FROM `wiadomosci` WHERE `id` = ".$id." LIMIT 0,1"));
 
 
echo 'Od: <b>'.$q[od_kogo].'</b> Nadana:<b>'.$q[time].'</b><br />';
echo 'Temat: <b>'.$q[temat].'</b><br />';
echo ''.$q[wiadomosc].'';
echo '<a href="panel.php?gog=poczta&wiad=odpowiedz'.$od_kogo.'"><br />Odpowiedz NA WIADOMOSC</a>';
?>
[PHP] pobierz, plaintext 

Uwaga: nie sprawdzane...

[pijanyadmin]

emm pozmieniałem z id na id_wiad żeby się nie gubić i teraz mam coś takiego:

[PHP] pobierz, plaintext 
<?php
$num=$_GET['num'];
$ids=$_SESSION['wiersze'][$num]['id_wiad'];
$od_kogo=$_SESSION['wiersze'][$num]['od_kogo'];
 
$q = mysql_fetch_array(mysql_query("SELECT * FROM `wiadomosci` WHERE `id_wiad` = '.$ids.' LIMIT 0,1"));
 
 
echo 'Od: <b>'.$q[od_kogo].'</b> Nadana:<b>'.$q[time].'</b><br />';
echo 'Temat: <b>'.$q[temat].'</b><br />';
echo ''.$q[wiadomosc].'';
echo '<a href="npanel.php?gog=poczta&wiad=odpowiedz'.$od_kogo.'"><br />Odpowiedz NA WIADOMOSC</a>';
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
while ($wiersz = mysql_fetch_array($quer)) 
{
$i++;
$class = ($i%2==1) ? "red" : "czarny";
echo '<div class="'. $class .'";>';
 
unset($_SESSION['wiersze']);
$_SESSION['wiersze'][$i]['id_wiad']=$wiersz['id_wiad'];
$_SESSION['wiersze'][$i]['od_kogo']=$wiersz['od_kogo'];
 
echo '<div class="pok"><a href="npanel.php?gog=poczta&wiad=odczyt&num='.$i.'">'.$wiersz[od_kogo].'</a></div>';
echo '<div class="pok2"> | <a href="npanel.php?gog=poczta&pocz=kasuj&num='.$i.'">usuń</a></div>';
echo '<div class="pok2"> | '.$wiersz[time].'</div>';
echo '<div class="pok2">'.$wiersz[temat].'</div>';
echo '</div>';
}
?>
[PHP] pobierz, plaintext 

i żadna z wiadomośći nie jest wczytywana, choć w linkach działa poprawnie

Ten post edytował pijanyadmin 3.02.2008, 13:00:14

[nowotny]

emm pozmieniałem z id na id_wiad żeby się nie gubić i teraz mam coś takiego:

[PHP] pobierz, plaintext 
<?php
$num=$_GET['num'];
$ids=$_SESSION['wiersze'][$num]['id_wiad'];
$od_kogo=$_SESSION['wiersze'][$num]['od_kogo'];
 
$q = mysql_fetch_array(mysql_query("SELECT * FROM `wiadomosci` WHERE `id_wiad` = '.$ids.' LIMIT 0,1"));
echo 'Od: <b>'.$q[od_kogo].'</b> Nadana:<b>'.$q[time].'</b><br />';
echo 'Temat: <b>'.$q[temat].'</b><br />';
echo ''.$q[wiadomosc].'';
echo '<a href="npanel.php?gog=poczta&wiad=odpowiedz'.$od_kogo.'"><br />Odpowiedz NA WIADOMOSC</a>';
?>
[PHP] pobierz, plaintext 

i żadna z wiadomośći nie jest wczytywana, choć w linkach działa poprawnie

Zapomniałeś session_start();...

[pijanyadmin]

nie zapomniałem sesja jest jak najbardziej, tylko wkleiłem sam kod w tkórym dokonałem zmian

[nowotny]

nie zapomniałem sesja jest jak najbardziej, tylko wkleiłem sam kod w tkórym dokonałem zmian

No to zobacz co ci wyświetla print_r($_SESSION['wiersze']);

[pijanyadmin]

Kod

Array ( [4] => Array ( [id_wiad] => 44 [od_kogo] => ggg ) )

ggg to jest ostatnia wiadomosc od uzytkownika...

[nowotny]

No to nie wiem... kombinuj i szukaj błędu...


BTW: tu jest literówka czy tak ma być:
$quer = mysql_query("SELECT * FROM `wiadomosci` WHERE `do_kogo` = '$login' ORDER BY `id` DESC");

[pijanyadmin]

jeszcze raz, nie poddałem się i prawie jestem chyba blisko...

[PHP] pobierz, plaintext 
<?php
session_start();
require("connect.php");
 
$quer = mysql_query("SELECT * FROM `mail` WHERE `odbiorca` = '$login' ORDER BY `id_mail` DESC");
 
$odbiorca=$_SESSION['odbiorca'];
$id_mail=$_SESSION['id_mail'];
 
while ($wiersz = mysql_fetch_array($quer)) 
{
$i++;
$class = ($i%2==1) ? "red" : "czarny";
echo '<div class="'. $class .'";>';
echo '<div class="pok"><a href="panel.php?gog=mail&wiad=odczyt&'.$odbiorca.'&id='.$id_mail.'">'.$wiersz[nadawca].'</a></div>';
echo '<div class="pok2"> | <a href="panel.php?gog=mail&mg=kasuj&id='.$id_mail.'">usuń</a></div>';
echo '<div class="pok2"> | '.$wiersz[time].'</div>';
echo '<div class="pok2">'.$wiersz[temat].'</div>';
echo '</div>';
}
?>
[PHP] pobierz, plaintext 

wyświetla wiadomość bez problemu, jednak schody zaczynają się przy wyświetleniu konkretnej wiadomośći, czyli po klikniecu w link:

[PHP] pobierz, plaintext 
<?php
session_start();
require("connect.php");
 
$q = mysql_fetch_array(mysql_query("SELECT * FROM `mail` WHERE `id_mail` = ".$_SESSION['id_mail']." LIMIT 0,1"));
 
echo 'Od: <b>'.$q[nadawca].'</b> Nadana:<b>'.$q[time].'</b><br />';
echo 'Temat: <b>'.$q[temat].'</b><br />';
echo ''.$q[wiadomosc].'';
 
?>
[PHP] pobierz, plaintext 

problem jest z zapytaniem do bazy o wyświetlenie konkretnej wiadomości... wpisałem już wszystkie kombinacje z sesji i zmiennych i... nic.

Ten post edytował pijanyadmin 4.02.2008, 02:10:23

[nowotny]

Spróbuj dostawić ciapki:

[PHP] pobierz, plaintext 
<?php
$q = mysql_fetch_array(mysql_query("SELECT * FROM `mail` WHERE `id_mail` = '".$_SESSION['id_mail']."' LIMIT 0,1"));
?>
[PHP] pobierz, plaintext 

$_SESSION['id_mail'] w ogóle zawiera poprawne dane...?

BTW, polecam się powoli przyzwyczajać do stosowania pojedyńczych cudzysłowów jak się odwołujesz do klucza tablicy... $wiersz['time'] zamiast $wiersz[time]...

Ten post edytował nowotny 4.02.2008, 03:03:03

[pijanyadmin]

Baza

[SQL] pobierz, plaintext 
CREATE TABLE IF NOT EXISTS `mail` (
  `id_mail` int(11) NOT NULL AUTO_INCREMENT,
  `nadawca` varchar(15) NOT NULL,
  `odbiorca` varchar(32) NOT NULL,
  `stan` varchar(40) NOT NULL,
  `temat` varchar(40) NOT NULL,
  `wiadomosc` varchar(255) NOT NULL,
  `time` varchar(19) NOT NULL,
  KEY `id_mail` (`id_mail`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 ;
[SQL] pobierz, plaintext 

[SQL] pobierz, plaintext 
INSERT INTO `mail` (`id_mail`, `nadawca`, `odbiorca`, `stan`, `temat`, `wiadomosc`, `time`) VALUES 
(1, 'nick1', 'nick2', '1', 'test', 'trescwiadomosciwyslanej', '10:47:21 04/02/2008');
[SQL] pobierz, plaintext 

jak na moje oko wszytko jest ok?

zmiana

Kod

'".$_SESSION['id_mail']."'

nie pomogła... wczytywany jest drugi plik ale nie są odczytywane dane poprzez właśnie sesje... tak jakby jej nie było...

Kod

echo 'Od: <b>'.$q[nadawca].'</b> Nadana:<b>'.$q['time'].'</b><br />';
echo 'Temat: <b>'.$q['temat'].'</b><br />';
echo ''.$q['wiadomosc'].'';

pozmieniałem to też z " " '' itp. ale niewiele to dało

[nowotny]

Czy to co jest 2 posty wyżej to jest cały skrypt...? Skąd bierzesz to:

[PHP] pobierz, plaintext 
<?php
$odbiorca=$_SESSION['odbiorca'];
$id_mail=$_SESSION['id_mail'];
?>
[PHP] pobierz, plaintext 

skoro nigdzie tego nie ustawiasz...?

[pijanyadmin]

w teorii to mają być dane... emm racja troche głupota... wczytuje dane przez

Kod

$quer = mysql_query("SELECT * FROM `mail` WHERE `odbiorca` = '$login' ORDER BY `id_mail` DESC");

wiadomo, baza, kto ma to odebrać na podstawie $login itp. To jak zapisać do linku dane odebrane z bazy?

[PHP] pobierz, plaintext 
<?php
echo '<div class="pok"><a href="panel.php?gog=mail&wiad=odczyt&'.$odbiorca.'&id='.$id_mail.'">'.$wiersz['nadawca'].'</a></div>';
?>
[PHP] pobierz, plaintext 

Ten post edytował pijanyadmin 4.02.2008, 11:33:29

[nowotny]

To u mnie działa... na samym początku podałeś inne nazwy kolumn i w ogóle inną tabele... :/ a potem ja i tak postawiłem w złym miejscu unseta...

[PHP] pobierz, plaintext 
<?php
session_start();
$quer = mysql_query("SELECT * FROM `mail` WHERE `odbiorca` = '$login' ORDER BY `id_mail` DESC");
 
unset($_SESSION['wiersze']);
while ($wiersz = mysql_fetch_array($quer)) 
{
	$i++;
	$class = ($i%2==1) ? "red" : "czarny";
	echo '<div class="'. $class .'";>';
 
 
	$_SESSION['wiersze'][$i]['id_wiad']=$wiersz['id_mail'];
	$_SESSION['wiersze'][$i]['od_kogo']=$wiersz['nadawca'];
 
	echo '<div class="pok"><a href="?gog=mail&wiad=odczyt&num='.$i.'">'.$wiersz['nadawca'].'</a></div>';
	echo '<div class="pok2"> | <a href="?gog=mail&mg=kasuj&num='.$i.'">usuń</a></div>';
	echo '<div class="pok2"> | '.$wiersz['time'].'</div>';
	echo '<div class="pok2">'.$wiersz['temat'].'</div>';
	echo '</div>';
}
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
session_start();
$num=$_GET['num'];
$ids=$_SESSION['wiersze'][$num]['id_wiad'];
$od_kogo=$_SESSION['wiersze'][$num]['od_kogo'];
 
$q = mysql_fetch_array(mysql_query("SELECT * FROM `mail` WHERE `id_mail` = ".$ids." LIMIT 0,1"));
 
echo 'Od: <b>'.$q[nadawca].'</b> Nadana:<b>'.$q['time'].'</b><br />';
echo 'Temat: <b>'.$q['temat'].'</b><br />';
echo ''.$q['wiadomosc'];
?>
[PHP] pobierz, plaintext 

Z resztą, nie rozumiem po jakiego grzyba pierwotnie przesyłałeś nazwę użytkownika w URLu... jak masz id to resztę możesz sobie wyciągnąć z bazy... dodatkowo to dosyć niebezpieczne rozwiązanie...

Aaa, i przy okazji: datę do bazy polecam sobie zapisywać w postaci uniksowego timestampa albo w tym Mysqlowym formacie... to co teraz masz bedzie bardzo trudno obrabiać...

Ten post edytował nowotny 4.02.2008, 11:51:05

[pijanyadmin]

Z resztą, nie rozumiem po jakiego grzyba pierwotnie przesyłałeś nazwę użytkownika w URLu... jak masz id to resztę możesz sobie wyciągnąć z bazy... dodatkowo to dosyć niebezpieczne rozwiązanie...

Z prostego powodu, mam problem ze zmianą sesji aby na podstawie id, były odczytywana reszta danych, teraz jest tak:

[PHP] pobierz, plaintext 
<?php
if(isset($_POST['user']) && isset($_POST['pass'])) {
	$login = htmlspecialchars($_POST['user']);
	$haslo = md5(htmlspecialchars($_POST['pass']));
	$q = mysql_query("SELECT * FROM userdata WHERE haslo = '$haslo' AND login = '$login' LIMIT 1"); 
					if(!mysql_num_rows($q)) {
		echo 'Niepoprawne Login lub Hasło'; exit;
		return 0;
	}
 
	$_SESSION['logon'] = 1;
	$_SESSION['login'] = $_POST['user'];
	$_SESSION['password'] = $_POST['pass'];
	$_SESSION['ip'] = $_SERVER['REMOTE_ADDR'];
 
	return 1;
	}
 if(!isset($_SESSION['logon']) || !isset($_SESSION['login']) || !isset($_SESSION['password']) || !isset($_SESSION['ip'])) {
					//	echo "3";
	return 0;
	}
		$login = $_SESSION['login'];
	$password = $_SESSION['password'];
	$ip = $_SESSION['ip'];
 
	if($_SERVER['REMOTE_ADDR'] != $ip) {
				//	echo "4";
	return 0;
	}
 
 
 if($_SESSION['logon'] == "1") { 
		   return 1;
	} 
	else {
	return 0;
	}
?>
[PHP] pobierz, plaintext 

czyli odebranie danych z formularza, porównanie loginu i hasła z bazą, oraz przekierowanie do odpowiedniej storny gdzie:

[PHP] pobierz, plaintext 
<?php
session_start();
require_once("functions.php");
require_once("connect.php");
 
if(!logon()) {
	resetsession();
	movetosite('index.php?sdd=logowanie');
	exit();
}
 
 
$login = $_SESSION['login'];
 
$query = mysql_query("SELECT * FROM userdata WHERE login = '$login'");
$dane = mysql_fetch_array($query);
$login = $dane[login];
$email = $dane[email];
?>
[PHP] pobierz, plaintext 

czyli na postawie loginu z sesji, porównanie go z bazą i odpowiednio przypisane dane do tego, w razie jakieś błedu/braku sesji przekierowanie do odpowieneij strony, w tym przypadku "'index.php?sdd=logowanie"

i poprostu nie wiem jak zamienic 

Kod

$login = $_SESSION['login'];

na

Kod

$ld = $_SESSION['id'];

a potem odczytać to jako

Kod

$query = mysql_query("SELECT * FROM userdata WHERE id = '$id'");

[nowotny]

A tak nie może być...?

[PHP] pobierz, plaintext 
<?php
if(isset($_POST['user']) && isset($_POST['pass'])) {
	$login = htmlspecialchars($_POST['user']);
	$haslo = md5(htmlspecialchars($_POST['pass']));
	$q = mysql_query("SELECT * FROM userdata WHERE haslo = '$haslo' AND login = '$login' LIMIT 1"); 
	if(mysql_num_rows($q)==FALSE) {
		echo 'Niepoprawne Login lub Hasło'; exit;
		return 0;
	}
	else{
		$userdata=mysql_fetch_assoc($q);
		$_SESSION['id']=$userdata['id']; //czy jaka tam masz nazwe dla kolumny z id usera
		$_SESSION['logon'] = 1;
		$_SESSION['login'] = $_POST['user'];
		$_SESSION['password'] = $_POST['pass'];
		$_SESSION['ip'] = $_SERVER['REMOTE_ADDR'];
 
			return 1;
	}
}
?>
[PHP] pobierz, plaintext 

[pijanyadmin]

emm a jak wtedy mam to odczytać po zalogowaniu?

[PHP] pobierz, plaintext 
<?php
if(!logon()) {
	resetsession();
	movetosite('index.php?sdd=logowanie');
	exit();
}
 
 
$userdata = $_SESSION['id'];
 
$query = mysql_query("SELECT * FROM userdata WHERE id = '$id'");
$dane = mysql_fetch_array($query);
$login = $dane[login];
$email = $dane[email];
?>
[PHP] pobierz, plaintext 

[nowotny]

No tak, nie...?

[PHP] pobierz, plaintext 
<?php
$userdata = $_SESSION['id'];
 
$query = mysql_query("SELECT * FROM userdata WHERE id = '$userdata'");
?>
[PHP] pobierz, plaintext 

Nie do końca rozumiem co masz na myśli... :/

[pijanyadmin]

emm ok mój błąd, tylko że teraz się namieszało troche... musze pozmieniać i dodać nową baze z nazwą użytkownika który wysłał wiadomość?

[PHP] pobierz, plaintext 
<?php
session_start();
$quer = mysql_query("SELECT * FROM `mail` WHERE `odbiorca` = '$userdata' ORDER BY `id_mail` DESC");
 
 
 
unset($_SESSION['wiersze']);
while ($wiersz = mysql_fetch_array($quer)) 
{
	$i++;
	$class = ($i%2==1) ? "red" : "czarny";
	echo '<div class="'. $class .'";>';
 
 
	$_SESSION['wiersze'][$i]['id_wiad']=$wiersz['id_mail'];
	$_SESSION['wiersze'][$i]['od_kogo']=$wiersz['nadawca'];
 
	echo '<img style="float:left; margin-top: 2px; margin-right: 2px;" src="'.$wiersz['stan'].'" alt="" /><div class="pok"><a href="?gog=mail&wiad=odczyt&num='.$i.'">'.$wiersz['nadawca'].'</a></div>';   
	echo '<div class="pok2"> | <a href="?gog=mail&mg=kasuj&num='.$i.'">usuń</a></div>';
	echo '<div class="pok2"> | '.$wiersz['time'].'</div>';
	echo '<div class="pok2">'.$wiersz['temat'].'</div>';
	echo '</div>';
}
?>
[PHP] pobierz, plaintext 

aby link i odczyt był prawidłowy?

[PHP] pobierz, plaintext 
<?php
session_start();
$num=$_GET['num'];
$ids=$_SESSION['wiersze'][$num]['id_wiad'];
$od_kogo=$_SESSION['wiersze'][$num]['od_kogo'];
 
$q = mysql_fetch_array(mysql_query("SELECT * FROM `mail` WHERE `id_mail` = ".$ids." LIMIT 0,1"));
 
echo 'Od: <b>'.$q[nadawca].'</b> Nadana:<b>'.$q['time'].'</b><br />';
echo 'Temat: <b>'.$q['temat'].'</b><br />';
echo ''.$q['wiadomosc'];
 
$z = mysql_query("UPDATE `mail` SET `stan` = 'go/mail/2.png' WHERE `id_mail` = ".$ids."");
?>
[PHP] pobierz, plaintext 

Ten post edytował pijanyadmin 4.02.2008, 20:45:24