Re[2]: [patch] bugfix for strip_password



Hello Roland,

Sunday, September 19, 2004, 6:36:16 PM, you wrote:

RI> Jindrich Makovicka wrote:
>> Hi,
>> 
>> In current mc, strip_password misbehaves for URLs like
>> 
>> /#ftp:user:pass hostname/path/path/path
>> 
>> which makes mc display only the string in the form
>> 
>> /#ftp:user hostname
>> 
>> in the title bar, instead of (IMHO correct)
>> 
>> /#ftp:user hostname/path/path/path
>> 
>> The attached patch attempts to re-implement the function with the 
>> correct behavior.

RI> I wrote a unit test for strip_password, and both versions (the old one
RI> and yours) translate the string as they should. Therefore I don't see a
RI> need to apply your patch.

    Please, test login as
    pa da ru or
    /#ftp:pavelsh da ru:pass hostname/path/path/path

-- 
Best regards,
 pavelsh                            mailto:pavelsh mail ru




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]