shout icon indicating copy to clipboard operation
shout copied to clipboard

Reconnection support and command/join order fix.

Open bews opened this issue 9 years ago • 7 comments

Based on code from #541.

Changes: exponential reconnection attempts fixed; added reconnect on ping timeout.

Every time reconnect happens, a message is posted to the main window:

2016-05-11_02-21-47

Issues: #658 #235 #395 #654

bews avatar May 10 '16 23:05 bews

Added fix for #662 - it changes already modified code, so I committed it here.

Problem was in irc.once("pong", function() { - PING command is firing from somewhere else too (besides irc.write("PING " + network.host);) so this event happens before user commands are finished.

bews avatar May 12 '16 00:05 bews

Merged all my additions in https://github.com/bews/shout/tree/bews-master - you can grab them from there till this is not merged.

bews avatar May 12 '16 02:05 bews

If it won't be merged, why not use TheLounge and contribute fixes for others to also use?

MaxLeiter avatar May 12 '16 04:05 MaxLeiter

If it won't be merged, why not use TheLounge and contribute fixes for others to also use?

As I said in previous PR:

when I started working with Shout I didn't know about Lounge fork, so I'll stick with this one :)

I just upload changes I did some time ago and don't plan making anything more here.

TheLounge is basically the same atm, so I don't feel like moving to it now. Maybe later, when it will have DCC and ban lists :)

bews avatar May 12 '16 04:05 bews

TheLounge is basically the same atm, so I don't feel like moving to it now.

Just closing the loop and not trolling here but I felt the need to point readers of this to the dedicated page listing the differences :smile: There is a lot in it, even though I admit lots are minor. Next release will be huge however and there is more to come before we release. We'll probably go through the PRs you opened to integrate what isn't there already (colors in action messages is fixed for example).

Thanks for your contributions to Shout anyway, and feel free to join the #shout-irc and #thelounge IRC channels! :-)

astorije avatar May 12 '16 07:05 astorije

There is a lot in it, even though I admit lots are minor.

But it still doesn't have the most important feature - auto reconnection :)

We'll probably go through the PRs you opened to integrate what isn't there already

This would be really good.

bews avatar May 14 '16 04:05 bews

But it still doesn't have the most important feature - auto reconnection :)

I knooow... So soon though: https://github.com/thelounge/lounge/pull/254

astorije avatar May 15 '16 20:05 astorije