Re: LiveUser::decryptPW

From: Date: Tue, 30 May 2006 11:19:42 +0000
Subject: Re: LiveUser::decryptPW
References: 1 2 3  Groups: php.pear.dev 
Request: Send a blank email to pear-dev+get-42688@lists.php.net to get a copy of this message
Matthias Nothhaft wrote:
Lukas Smith wrote:
Matthias Nothhaft wrote:
Hi, in CVS I believe decryptPW should "return $encryptedPW;" at the end of the function instead of $decryptedPW Furthermore $plainPW is not set but returned.. So someone should probably check this again?
thxy for the heads up! please check CVS .. i just commited some improvements ..
hm.. in my good old LU version md5 is returned as it is, so why do you push an error in this case? It it a new concept to not use decryption if not possible?
Not really a "concept" .. But I think its cleaner and you can easily handle the error in userland, where as it was previously not possible to determine if the password was actually decrypted or not. regards, Lukas

« previous php.pear.dev (#42688) next »