[dev] Registry.php: Ciphertext must not be empty

[Michael M Slusarz]

Quoting Jan Schneider <jan at horde.org>:

> Zitat von "M. Braun" <mi.braun at onlinehome.de>:
>
>> Hi,
>>
>> when trying to setup horde 4 from git, a got an error stateting that the
>> ciphertext must not be empty from Crypt_Bluefish.
>> It turned out that the $_SESSION['horde_auth']['app'] array exists
>> but was empty, so no ciphertext for the $app was saved.
>>
>> This could be fixed by
>> *** Registry.php.orig   2010-06-21 08:34:38.000000000 +0200
>> --- Registry.php        2010-06-21 08:03:53.000000000 +0200
>> *************** class Horde_Registry
>> *** 1915,1921 ****
>>       */
>>      protected function _getAuthCredentials($app)
>>      {
>> !         if (!isset($_SESSION['horde_auth']['app'])) {
>>              return false;
>>          }
>>
>> --- 1915,1921 ----
>>       */
>>      protected function _getAuthCredentials($app)
>>      {
>> !         if (!isset($_SESSION['horde_auth']['app'][$app])) {
>>              return false;
>>          }
>>
>> where Registry.php is horde/libs/Registry.php.
>
> This sounds suspiciously like curing the symptoms instead of fixing  
> the cause. I'm not as deep into the authentication code as Michael,  
> but I guess the array shouldn't be empty in the first place.

This doesn't look right.  Your auth setup is either something that I  
have not tested, or there is something screwy about your local setup.   
You are doing just what Jan said - curing the symptom, not the problem  
(i.e. turning up the car radio to mask the weird noise coming from  
underneath the hood).

michael

-- 
___________________________________
Michael Slusarz [slusarz at horde.org]