LiveUser::decryptPW

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?

Regards,
Matthias

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub
.php

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

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub
.php