Convos locks up after connecting to specific IRC Server

[jsuelwald]

Describe the bug
The convos server kindof freezes after a while after being connected to specific irc-servers
I think the convos server dies when trying to retrieve a Channel list on that server.

Reason for this belive is this server Response: "Usage of /list for listing all channels is deprecated. Please use "/squery alis help" instead."

To Reproduce
Steps to reproduce the behavior:

1. Add a connection to irc.uni-erlangen.de
2. Try to join a channel

Expected behavior
Should be able to get a channel list and join a channel.

Screenshots
Log:

[2021-02-23 08:02:55.81724] [8645] [info] Listening at "http://0.0.0.0:8080"
Web application available at http://0.0.0.0:8080
Argument "&" isn't numeric in addition (+) at /root/convos/lib/Convos/Core/Connection/Irc.pm line 398.
Argument "i" isn't numeric in addition (+) at /root/convos/lib/Convos/Core/Connection/Irc.pm line 398.
Argument "nuby" isn't numeric in addition (+) at /root/convos/lib/Convos/Core/Connection/Irc.pm line 398.
Argument "dig" isn't numeric in addition (+) at /root/convos/lib/Convos/Core/Connection/Irc.pm line 398.
Use of uninitialized value in substitution (s///) at /root/convos/lib/Convos/Core/Connection/Irc.pm line 402.
Mojo::Reactor::EV: I/O watcher failed: Can't use an undefined value as an ARRAY reference at /root/convos/lib/Convos/Core/Connection/Irc.pm line 205.
Use of uninitialized value in pattern match (m//) at /root/convos/lib/Convos/Core/Connection/Irc.pm line 872.
Use of uninitialized value in pattern match (m//) at /root/convos/lib/Convos/Core/Connection/Irc.pm line 872.
Use of uninitialized value in pattern match (m//) at /root/convos/lib/Convos/Core/Connection/Irc.pm line 872.
Mojo::Reactor::EV: I/O watcher failed: Can't use an undefined value as an ARRAY reference at /root/convos/lib/Convos/Core/Connection/Irc.pm line 205.
Use of uninitialized value in pattern match (m//) at /root/convos/lib/Convos/Core/Connection/Irc.pm line 872.
Use of uninitialized value in pattern match (m//) at /root/convos/lib/Convos/Core/Connection/Irc.pm line 872.
Use of uninitialized value in pattern match (m//) at /root/convos/lib/Convos/Core/Connection/Irc.pm line 872.
Use of uninitialized value in pattern match (m//) at /root/convos/lib/Convos/Core/Connection/Irc.pm line 872.
Use of uninitialized value in pattern match (m//) at /root/convos/lib/Convos/Core/Connection/Irc.pm line 872.
Argument "NOTICE" isn't numeric in addition (+) at /root/convos/lib/Convos/Core/Connection/Irc.pm line 398.
Argument "NOTICE" isn't numeric in addition (+) at /root/convos/lib/Convos/Core/Connection/Irc.pm line 398.
Argument "NOTICE" isn't numeric in addition (+) at /root/convos/lib/Convos/Core/Connection/Irc.pm line 398.
Argument "NOTICE" isn't numeric in addition (+) at /root/convos/lib/Convos/Core/Connection/Irc.pm line 398.

Environment:

• OS: Ubuntu Server 20.04 LTS
• Browser Vivaldi (Current)
• Version: Latest stable as of 2021-02-23
• IRC server irc.uni-erlangen.de - Version;: 2.11.2p3. Uni-Erlangen.DE 276E aCEFHJKlmMpQRsTuZ6

[jhthorsen]

When you say "Try to join a channel" does that mean that you go to "Add conversation" in the sidebar, or running "/join #channelname" ?

Also, which IRC server software does irc.uni-erlangen.de use?

[jsuelwald]

Using /JOIN works.
Using Add Conversation: Convos dies.

IRC server irc.uni-erlangen.de - Version;: 2.11.2p3. Uni-Erlangen.DE 276E aCEFHJKlmMpQRsTuZ6

[jhthorsen]

I'm thinking this should be fixed in some/all of these places:

• convos/lib/Convos/Core/Connection/Irc.pm

  Line 914 in 37a0ade

  $self->_write("LIST\r\n");

• convos/lib/Convos/Core/Connection/Irc.pm

  Lines 392 to 405 in 37a0ade

  	sub _irc_event_rpl_list {
  	my ($self, $msg) = @_;
  	my $conversation = {n_users => 0 + $msg->{params}[2], topic => $msg->{params}[3]};
  	$conversation->{name} = $msg->{params}[1];
  	$conversation->{conversation_id} = lc $conversation->{name};
  	$conversation->{topic} =~ s!^(\[\+[a-z]+\])\s?!!; # remove mode from topic, such as [+nt]
  	$self->_available_conversations->{conversations}{$conversation->{name}} = $conversation;
  	}
  	sub _irc_event_rpl_listend {
  	my ($self, $msg) = @_;
  	$self->_available_conversations->{done} = true;
  	}

But unfortunately I have no idea how to fix it. Running convos with CONVOS_LOG_LEVEL=trace would give som valuable output.

Going to close this issue for now, since I think it has gone stale.