Bug 254413 - Quit msg does not work when using ssl
Summary: Quit msg does not work when using ssl
Status: RESOLVED FIXED
Alias: None
Product: konversation
Classification: Applications
Component: general (show other bugs)
Version: 1.3.1
Platform: Debian testing Linux
: NOR normal
Target Milestone: ---
Assignee: Konversation Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-10-16 22:34 UTC by Carlos
Modified: 2011-06-30 23:20 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Carlos 2010-10-16 22:34:29 UTC 
Version:           1.3.1 (using KDE 4.4.5) 
OS:                Linux

When quiting konversation, or when using /quit foo, other user do not see the quit msg.
freenode: (Remote host closed the connection)
other servers i testet: (Input/Output error)

I can remember quit msg working on konversation, but i dont know with what version it stopt working

Reproducible: Always

Steps to Reproduce:
quit the server
Ctrl + q
/quit foo

Actual Results:  
Looks like konversation does not make a clean disconnect with the server:
(Input/Output error)

Expected Results:  
other people should see the quit msg
Comment 1 Carlos 2010-10-17 00:42:36 UTC 
Ok, after further testing it does look like quit msg does not work when ssl is enabled.
Can someone change the to: "Quit msg does not work when using ssl" ?

Thanks
Comment 2 Eike Hein 2011-06-27 12:31:51 UTC 
Carlos, could you try if this works for you in git master? There were some changes to quit message handling a few weeks ago.
Comment 3 Carlos 2011-06-27 12:43:49 UTC 
I will try.
But can no one of you see that error? Is that only me?

Carlos
Comment 4 Eike Hein 2011-06-27 13:18:38 UTC 
No, there was a more or less reproducable race condition in the quit message handling that prompted the changes, we'd just like some independent confirmation that things are fixed now.
Comment 5 Carlos 2011-06-30 23:15:52 UTC 
Sweet. It works now. Thanks!
Comment 6 Eike Hein 2011-06-30 23:20:16 UTC 
Good :). Thanks for confirming!