Closed
Bug 788137
Opened 12 years ago
Closed 10 years ago
Nick for XMPP chatrooms becomes 'null'
Categories
(Thunderbird :: Instant Messaging, defect)
Thunderbird
Instant Messaging
Tracking
(thunderbird31 fixed, thunderbird32 fixed, thunderbird33 fixed)
VERIFIED
FIXED
Thunderbird 33.0
People
(Reporter: marcus.ilgner, Assigned: aleth)
References
Details
Attachments
(1 file, 1 obsolete file)
2.63 KB,
patch
|
florian
:
review+
standard8
:
approval-comm-aurora+
standard8
:
approval-comm-beta+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 6.2; WOW64) AppleWebKit/537.1 (KHTML, like Gecko) Chrome/21.0.1180.89 Safari/537.1 Steps to reproduce: Create XMPP account Choose "Join Chat" Enter room, server and a custom nickname Select 'Auto-join this chat room' Actual results: Upon reconnecting, the nickname is set to 'null'. In the accounts dialog, there's a "/null" at the end of the room. Simply replacing this did not seem to fix the problem. Expected results: The nick that was previously specified should have been saved. Nicks containing spaces should be supported, too. (I tried both variants)
Reporter | ||
Comment 1•12 years ago
|
||
Just tried it with the Earlybird release. Changing the nickname at the end of the channel name in the accounts dialog seems to work now. It is still set to 'null' when first joining the channel though.
Comment 2•12 years ago
|
||
I just tried to reproduce this and couldn't. Can you still reproduce this? If you can, could you please give more detailed steps to reproduce? Thanks!
I have the same issue with 17.05. When I first join everything is fine, but several hours later I become "null" in the room. The only way to fix is to leave the chat room and reconnect, and set the nickname again. I haven't even disconnected from the chat room when this happens.
Same issue. Very annoying. It is working well with pidgin. Please, stop adding features to Thunderbird, and fix bugs first.
Comment 6•10 years ago
|
||
(In reply to Andre Rodier from comment #5) > Same issue. Very annoying. Could you please provide detailed steps to reproduce the issue?
Comment 7•10 years ago
|
||
Same here. Steps to reproduce: 1. Create jabber account 2. Connect 3. join chat, fill parameters, check join on connect checkbox The /null configuration is now persisted.
Assignee | ||
Comment 8•10 years ago
|
||
(In reply to Thorsten S. from comment #7) > Same here. Steps to reproduce: > > 1. Create jabber account > 2. Connect > 3. join chat, fill parameters, check join on connect checkbox > > The /null configuration is now persisted. Which server are you using? (Of course, if you don't wish to share that information, that's OK, but it would help to reproduce this.) Does this only happen if you pick a different nick for the chatroom than your XMPP name (what's in front of the @ in your JID) or doesn't that make any difference? Are there any special characters in your nick?
Comment 9•10 years ago
|
||
Same bug with debianforum.de:5222 and our local company jabber server. The nick can differ or not, the nick part will be null (tested again). My nick(s) consists of 4-8 simple ascii characters, tested mixed case and lower case.
Assignee | ||
Updated•10 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Assignee | ||
Updated•10 years ago
|
Component: Instant Messaging → XMPP
OS: Windows 8 → All
Product: Thunderbird → Chat Core
Hardware: x86_64 → All
Version: 15 → trunk
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → aleth
Assignee | ||
Comment 10•10 years ago
|
||
STR: Join an XMPP MUC from the Join Chat dialog, checking the autojoin box. Then look at the autojoin pref value. The nick is always saved as "null". I'll add a port for TB if you agree with this fix. (I dislike it but can't think of anything better.)
Attachment #8448100 -
Flags: feedback?(clokep)
Assignee | ||
Comment 11•10 years ago
|
||
Forgot to ask: I think we could do without the protoId checks altogether here (now we check for the presence of the value), which would make this less ugly. Am I overlooking something?
Comment 12•10 years ago
|
||
So I looked at this briefly...this is pretty hacky. I think the right way to do this would be to add an nsISimpleEnumerator of prplIChatRoomFieldValues to the imIAccount (not the prplIAccount, just to be clear) under a field called "autoJoinRooms" or something. This could then be serialized to disc and reloaded each time. This would allow us to be "smarter" about what we actually save from each joined channel. Note that this would also mean the auto-join field is no longer just a string...so that's quite a bit more of an invasive change (but would be a huge jump toward session restore). I have to think about whether we'd want to accept this hack or not, so this definitely isn't an f-, but more of the "perfect" way to do it.
Assignee | ||
Comment 13•10 years ago
|
||
I noticed we return early if the prpl isn't irc, jabber or gtalk anyway, so my idea in comment 11 is ok to make this slightly less ugly. I agree with comment 12 in that this isn't the ultimate solution, but it fixes the bug with the existing autojoin mechanism. Ceterum censeo autojoin esse delendam, etc. Should this be uplifted to TB31?
Attachment #8448100 -
Attachment is obsolete: true
Attachment #8448100 -
Flags: feedback?(clokep)
Attachment #8448615 -
Flags: review?(clokep)
Comment 14•10 years ago
|
||
Comment on attachment 8448615 [details] [diff] [review] nullnick.diff 2 (In reply to aleth [:aleth] from comment #13) > Should this be uplifted to TB31? Yes, please :-).
Attachment #8448615 -
Flags: review?(clokep) → review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 15•10 years ago
|
||
https://hg.mozilla.org/comm-central/rev/92d808571804
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.6
Comment 16•10 years ago
|
||
aleth: We want to request approval for beta on this, right?
Keywords: checkin-needed
Assignee | ||
Comment 17•10 years ago
|
||
(In reply to Patrick Cloke [:clokep] from comment #16) > aleth: We want to request approval for beta on this, right? Yes. It would be nice if one of the bug reporters could confirm this fixes the problem for them as well (with the next Instantbird nightly or TB Daily build).
Comment 18•10 years ago
|
||
(In reply to aleth [:aleth] from comment #17) > It would be nice if one of the bug reporters could confirm this fixes the > problem for them as well (with the next Instantbird nightly or TB Daily > build). Which versions of the below linked, and when? "Daily" has no checkins in this week, so it says. I'd volunteer. [1] http://forums.mozillazine.org/viewtopic.php?f=29&t=2850329
Comment 19•10 years ago
|
||
Yes, you want the latest Daily: http://ftp.mozilla.org/pub/mozilla.org/thunderbird/nightly/latest-comm-central/
Comment 20•10 years ago
|
||
works for me, thanks a lot for your commitment.
Assignee | ||
Updated•10 years ago
|
Component: XMPP → Instant Messaging
Product: Chat Core → Thunderbird
Target Milestone: 1.6 → ---
Version: trunk → Trunk
Assignee | ||
Comment 22•10 years ago
|
||
Comment on attachment 8448615 [details] [diff] [review] nullnick.diff 2 [Approval Request Comment] User impact if declined: This is an annoying bug for anyone using XMPP autojoins. Testing completed (on c-c, etc.): Verified fixed Risk to taking this patch (and alternatives if risky): Low risk
Attachment #8448615 -
Flags: approval-comm-beta?
Attachment #8448615 -
Flags: approval-comm-aurora?
Comment 23•10 years ago
|
||
(In reply to aleth [:aleth] from comment #22) > [Approval Request Comment] > User impact if declined: This is an annoying bug for anyone using XMPP > autojoins. This bug has been reported several times by several different people since we initially released Chat in Thunderbird. Now that we have identified the cause and have a simple fix, I think we should ship it in the next Thunderbird release.
Updated•10 years ago
|
Attachment #8448615 -
Flags: approval-comm-beta?
Attachment #8448615 -
Flags: approval-comm-beta+
Attachment #8448615 -
Flags: approval-comm-aurora?
Attachment #8448615 -
Flags: approval-comm-aurora+
Assignee | ||
Comment 24•10 years ago
|
||
https://hg.mozilla.org/releases/comm-aurora/rev/8a344d493bfc https://hg.mozilla.org/releases/comm-beta/rev/d350cf2e26fd
Assignee | ||
Updated•10 years ago
|
Target Milestone: --- → Thunderbird 31.0
Updated•10 years ago
|
status-thunderbird31:
--- → fixed
status-thunderbird32:
--- → fixed
status-thunderbird33:
--- → fixed
Target Milestone: Thunderbird 31.0 → Thunderbird 33.0
You need to log in
before you can comment on or make changes to this bug.
Description
•