Closed
Bug 783457
Opened 12 years ago
Closed 12 years ago
about:support uses hostname instead of realhostname
Categories
(Thunderbird :: General, defect)
Thunderbird
General
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 17.0
People
(Reporter: jcranmer, Assigned: aceman)
References
Details
Attachments
(1 file)
1.11 KB,
patch
|
mconley
:
review+
|
Details | Diff | Splinter Review |
I've always hated the names of these two attributes, since it means people are probably going to be very apt to pick the wrong one.
Looks like this line should be changed: http://hg.mozilla.org/comm-central/file/3b75111da3ba/mail/components/about-support/aboutSupport.js#l83
OS: Windows 7 → All
Hardware: x86_64 → All
It works for me on Linux.
While testing it out I noticed my IM account (of type "JS test") shows garbage as: (im) realHostName FIXME:0 Probably realHostName is unimplemented. Also the port is shown as 0. The about:support generator hides the port only when it is set to -1. That is used in Local Folders and Feeds account types. Florian, please look at it if this isn't a problem in more IM protocols (or just in JS test).
Version: unspecified → Trunk
Reporter | ||
Comment 4•12 years ago
|
||
Comment on attachment 652772 [details] [diff] [review] patch I'm not a peer of the FE stuff, so redirecting to mconley. That said, this looks good to me...
Attachment #652772 -
Flags: review?(Pidgeot18) → review?(mconley)
Comment 5•12 years ago
|
||
Comment on attachment 652772 [details] [diff] [review] patch Review of attachment 652772 [details] [diff] [review]: ----------------------------------------------------------------- It's kind of a tragedy that we have "hostName" and "realHostName". Puts a bad taste in my mouth. Anyhow, I haven't tested this, but I trust you have, and it looks like it's doing the right thing.
Attachment #652772 -
Flags: review?(mconley) → review+
I have tested the appearance on Linux and it works (changes the shown server host) and also changes the host in the IM account to the placeholder string. So it does something :)
Keywords: checkin-needed
Comment 7•12 years ago
|
||
(In reply to :aceman from comment #3) > While testing it out I noticed my IM account (of type "JS test") shows > garbage as: > (im) realHostName FIXME:0 > > Probably realHostName is unimplemented. Also the port is shown as 0. The > about:support generator hides the port only when it is set to -1. That is > used in Local Folders and Feeds account types. > Florian, please look at it if this isn't a problem in more IM protocols (or > just in JS test). It's not specific to any IM protocol (so no, this isn't just for JS test); this value is from imIncomingServer at http://mxr.mozilla.org/comm-central/source/mail/components/im/imIncomingServer.js#57 I couldn't find any use case for these 2 values at the time I wrote this code.
Comment 8•12 years ago
|
||
https://hg.mozilla.org/comm-central/rev/8261f4f34dca
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Flags: in-testsuite-
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 17.0
So it seems now there is a use case and the comment in the code is no longer valid. The values are exposed now.
You need to log in
before you can comment on or make changes to this bug.
Description
•