Comment on attachment 570960
Part 3. FTP directory list sets the encoding when current FTP server supports 
UTF-8 v2

> +   channel->GetContentCharset(mCharset);
> +   if (mCharset.EqualsLiteral("UTF-8")) {
> +       // For RFC 2640 support, charset is into HTML, not channel.
> +       // So we need clear charset on channel.
> +       channel->SetContentCharset(NS_LITERAL_CSTRING(""));
> +   }

Could you please make the comment more verbose? Why it is needed to
clear the charset and why only in case of UTF8?


> +   if (mCharset.EqualsLiteral("UTF-8") && !IsUTF8(mPendingBuffer)) {
> +       // RFC 2640 section 3.3 says the following.
> +       //
> +       // If a client detects that a server is non UTF-8,
> +       // it SHOULD change its display appropriately.
> +       //
> +       // So we need failback encoding.
> +       mCharset.AssignLiteral("ISO-8859-1");
> +   }

There is still a problem when the directory is empty and its name isn't
a valid UTF8 string. In this case call to
mTextToSubURI->UnEscapeAndConvert() in
nsIndexedToHTML::OnHeaderAvailable() fails.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/222635

Title:
  firefox ftp client don't understand links in cyrillic letters

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/222635/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to