961908 - Accounts with IDN in server names work poorly

Status: RESOLVED FIXED

(MailNews Core :: Account Manager, defect)

Description

[Joshua Cranmer [:jcranmer]]

I've been trying to set up an IDN/EAI testbed for Thunderbird development, and after several failed attempts to set up an account to my client, I found what appears to be a major IDN issue.

Given that I didn't trust the account dialog to accept U-labels, I tried the A-label, which for my server is xn--bckc7bj7f1a6vd.tlalocan. (Try having to retype that several times). This passes all the initial validations, and when we finally hit the VerifyLogin stage, it all of a sudden blows up. When we're in nsMsgDBFolder::ParseURL, we have:
(gdb) p mURI
$1 = {<nsACString_internal> = {mData = 0x7f8db5649108 "imap://[e-mail]", mLength = 48, mFlags = 5}, <No data fields>}

However, when we get to nsMsgAccountManager::FindServerByURI, we get:
(gdb) p hostname.get()
$5 = (const nsACString_internal::char_type *) 0x7fff35c072a0 "mail.ケツァルコアトル.tlalocan"

So nsStandardURL helpfully standardized the IDN for us (server->GetHostName() has the xn-- variant still), and we wind up being unable to match the URL anymore, causing the entire process to crash and burn.

Comment 1

[Magnus Melin [:mkmelin]]

I fixed this in bug 1870720 (and dependents).