[php] wygrawerowanie i przesłanie nowego hasła, problem Opcje

[Lejto]

Mam kod który graweruje nowe hasło updatuje do bazy przesyła je na danego maila.
Kod był wiele razy modyfikowany więc niektóre elementy mogą być niepotrzebne.
Raz kod zadziałał zupdatował nowe hasło do bazy ale nie wysłał mailem wiadomość doszła ale bez hasła (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)
Teraz ani to ani to nie działa.
Kody: (kody formularza nie podaje bo chyba nie ma po co)

[PHP] pobierz, plaintext 
<?php
tworz_naglowek_html('Ustawianie hasła');
 
  // utworzenie krótkiej nazwy zmiennej
  $user = $_POST['user'];
 
  try
  {
	 $haslo=ustaw_haslo($user);
	 powiadom_haslo($user, $haslo);
	 echo 'Nowe hasło zostało przesłane na adres poczty elektronicznej.<br />';
  }
  catch (Exception $e)
  {
	 echo 'Hasło nie mogło zostać ustawione. Proszę spróbować póĽniej.';
  }
  tworz_HTML_URL('?co=log', 'Logowanie');
  tworz_stopke_html();
?>
[PHP] pobierz, plaintext 

i grawerowanie

[PHP] pobierz, plaintext 
<?php
function pobierz_losowe_slowo($dlugosc_min, $dlugosc_max)
//pobranie losowego słowa ze słownika o okre?lonej długo?ci zwrócenie go
{
 global $haslo;  
 
$ile_liter = 4; // Ile liter ma mieć losowany ciąg  
$litery = 'qwertyuiopasdfghjklzxcvbnm1234567890'; // Zestaw liter z jakich ma być zbudowany ciąg  
 
// 'Zamieszanie komory losującej'  
// UWAGA: od wersji 4.2.0 PHP nie jest to juz konieczne  
srand((double)microtime()*1000000);  
 
$haslo = ''; // Ustawienie początkowe hasła  
for ($i=0;$i<$ile_liter;$i++)  
   { $haslo .= substr($litery,(rand()%(strlen($litery))),1); };  
}
 
 
 
function ustaw_haslo($user)
// ustawienie hasła użytkownika na losowš warto?ć 
// zwraca nowe hasło lub false w przypadku niepowodzenia
{ 
 echo $haslo;
  $wynik = mysql_query("update uzytkownik
						 set haslo = sha1('$haslo')
						 where user = '$user'");
  if (!$wynik)
   echo 'Zmiana hasła nie powiodła się.';  // hasło nie zmienione
  else
	return $haslo;  // hasło zmienione pomyślnie  
}
 
function powiadom_haslo($user, $haslo)
// powiadomienie użytkownika o zmianie hasła
{
 
	$wynik = mysql_query("select email from uzytkownik
						   where user='$user'");
	if ($wynik == false)
	{
	  echo 'Nie znaleziono adresu e-mail<br>';
	}
	else
	{
	  $wiersz = mysql_fetch_object($wynik);
	  $email = $wiersz->email;
	  $od = "From: admin@admin.pl r\n";
	  $wiad = "Hasło na stronie ddd zostało zmienione na:r\n"
	  ."--------------------r\n"
	  .$haslo
	  ."r\n--------------------r\n"
	  ."Proszę zmienić je przy następnym logowaniu. r\n"
	  ."W razie niedziałania hasła lub jakiego kolwiek błędu proszę powiadomić administratora strony r\n";
	  echo 'elo';
	  echo $haslo;
 
	  if (mail($email, 'Przesłanie nowego hasła', $wiad, $od))
		return true;	  
	  else
		echo 'Wysłanie e-maila nie powiodło się';
	}
}
?>
[PHP] pobierz, plaintext 

Ten post edytował Lejto 2.05.2008, 17:10:28

[marcio]

W funkcji pobierz_losowe_haslo masz global po co ono ma byc w ustaw haslo bo z tamdat pobierasz zmienna $haslo a ja tworzysz w 1 funckji wiec jest zbedne
Sproboj tak

[PHP] pobierz, plaintext 
<?php
function pobierz_losowe_slowo($max) { //tak nie szybciej i latwiej??  
 
$nowe_haslo = substr(md5(time()), 0, $max);
return $nowe_haslo;
}
 
 
function powiadom_haslo($user, $pass) {
 
if($wynik = mysql_query('select email from uzytkownik where user = "'.$user.'" limit 1')) {
 
$wiersz = mysql_fetch_assoc($wynik);
$email = $wiersz['email'];
$od = "From: admin@admin.pl r\n";
$wiad = 'Hasło na stronie 2c-nowe.com zostało zmienione na: '.$pass.'
Proszę zmienić je przy następnym logowaniu
W razie niedziałania hasła lub jakiego kolwiek błędu proszę powiadomić administratora strony';
	 
if(mail($email, 'Przesłanie nowego hasła', $wiad, $od)) echo('Email z nowym haslem zostal wyslany');	  
else echo('Wysłanie e-maila nie powiodło się');
  }
else echo('Nie znaleziono email'a');
}
 
 
function ustaw_haslo($user, $new_pass){ 
 
if($wynik = mysql_query('update uzytkownik set haslo = "'.sha1($new_pass).'" where user = "'.$user.'"')) echo('Zmieniono haslo')
else echo('Blad przy zmianie hasla');
 
}
 
$login = $_POST['login'];
 
$haslo = pobierz_losowe_haslo(8); // tu masz nowe haslo
ustaw_haslo($login, $haslo);
powiadom_haslo($login, $haslo);
?>
[PHP] pobierz, plaintext 

Powiedz ewentualnie co nie dziala pisalem z palca moze sa jakies bledy ogolnie tez w CMS tak zrobilem zapomniane haslo ale uwierz mi to nie jest dobry sposob zabezpiecz to sobie bo mozna walki robic chyba ze sprawdzasz dane z form'a ale nigdzie tu tego nie widze

W kodzie miales cos takiego

[PHP] pobierz, plaintext 
<?php
echo $haslo;
?>
[PHP] pobierz, plaintext 

W 2 funkcjach z zmienna $haslo tworzysz w funkcji pobierz_losowe_haslo() i to nie mialo prawa dzialac bo po
1.to u ciebie ta funkcja nic nie zwraca
2.to nie echo() ale jak juz ta zmienna ma byc widoczna w inne funckji to

[PHP] pobierz, plaintext 
<?php
global $haslo;
?>
[PHP] pobierz, plaintext 

3.

[PHP] pobierz, plaintext 
<?php
sha1('$haslo')
?>
[PHP] pobierz, plaintext 

Zmienna to nie string i chyba tak sie jej nie przekazuje

Ten post edytował marcio 2.05.2008, 13:56:06