[patch] bugfix for strip_password

pavelsh pavelsh at mail.ru
Sun Sep 19 20:07:15 UTC 2004


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 at hostname/path/path/path
>> 
>> which makes mc display only the string in the form
>> 
>> /#ftp:user at hostname
>> 
>> in the title bar, instead of (IMHO correct)
>> 
>> /#ftp:user at 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 at da.ru or
    /#ftp:pavelsh at da.ru:pass at hostname/path/path/path

-- 
Best regards,
 pavelsh                            mailto:pavelsh at mail.ru




More information about the mc-devel mailing list