1389173 - Implement nsLDAPURL::GetDisplay[Spec|Host|HostPort] better and enhance test

Status: RESOLVED FIXED

(MailNews Core :: LDAP Integration, enhancement)

Description

[Jorg K (CEST = GMT+2)]

+++ This bug was initially created as a clone of Bug #1388927 +++

In bug 1388927 I had to adjust the test expectation
https://hg.mozilla.org/comm-central/rev/430f8faf075db3d4915786d534b811530267fb23#l1.12
since spec, host, hostPort and prePath now return (ASCII) punycode due to bug 1380617.

Since the test contains a URL with Japanese characters we should consider implementing nsLDAPURL::GetDisplay[Spec|Host|HostPort] and restore the test case.

Comment 1

[Jorg K (CEST = GMT+2)]

Attachment: 1389173-ldap-url.patch (v1)

Comment 2

[Jorg K (CEST = GMT+2)]

Looking at bug 1389762, we might need to do something like this for NNTP as well.

Comment 3

[Jorg K (CEST = GMT+2)]

Please ignore the previous comment, bug 1389762 has a different cause.

Comment 4

[Jorg K (CEST = GMT+2)]

Attachment: 1389173-ldap-url.patch (v1) - rebased

Valentin, do you think it makes sense to "improve" the implementations of GetDisplay[Spec|Host|HostPort] or should this bug just be a WONTFIX?

Comment 5

[:aceman]

Can we use any of these Display* attributes in the base code?

Somehow we have a lot of reported problems (in TB52) due to urls being changed to punycode internally even before bug 1388927.
We still couldn't solve them :(

Comment 6

[Jorg K (CEST = GMT+2)]

(In reply to :aceman from comment #5)
> Can we use any of these Display* attributes in the base code?
What do you mean?

> Somehow we have a lot of reported problems (in TB52) due to urls being
> changed to punycode internally even before bug 1388927.
Reported where?

Comment 7

[:aceman]

(In reply to Jorg K (GMT+2) from comment #6)
> (In reply to :aceman from comment #5)
> > Can we use any of these Display* attributes in the base code?
> What do you mean?

If we call these properties anywhere in TB. It seems not. Only the test uses them now.
But as those attributes are defined on the parent classes we should have them report correct values (decoded).

> > Somehow we have a lot of reported problems (in TB52) due to urls being
> > changed to punycode internally even before bug 1388927.
> Reported where?

Like bug 1369343 and bug 1360836. I have debugged those locally and it is because at some point m-c coverts the international server names to punycode which we do not expect and then we can no longer find any folders for the server.

Comment 8

[Jorg K (CEST = GMT+2)]

(In reply to :aceman from comment #7)
> Like bug 1369343 and bug 1360836. I have debugged those locally and it is
> because at some point m-c coverts the international server names to punycode
> which we do not expect and then we can no longer find any folders for the
> server.
And we haven't fixed those yet :-(

Comment 9

[Jorg K (CEST = GMT+2)]

Should we consider implementing better GetDisplay[Spec|Host|HostPort|PrePath] for the other three URLs we have: mailnews, address book and mailto? See: https://hg.mozilla.org/comm-central/rev/4c712bcd9062.

For LDAP this bug here was motivated since I had to change a test here:
https://hg.mozilla.org/comm-central/rev/430f8faf075d

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/a75878fe1f57
Implement better GetDisplay[Spec|Host|HostPort|PrePath] for LDAP and enhance related test. r=aceman