Add an argument to ibis-client for a list of channels to join

Review Request #3606 — Created Oct. 19, 2024 and submitted

Information

ibis/ibis
default

Reviewers

I also plugged a bunch of memory leaks around the command line arguments and
reordered them as well.

Connected to a local ergo server and verified it joined the channels correctly.

Summary ID
Add an argument to ibis-client for a list of channels to join
I also plugged a bunch of memory leaks around the command line arguments and reordered them as well.
ce6882d55af1f888183d82e45b4c5a7b70d79ed1
Description From Last Updated

Should this be G_OPTION_ARG_STRING_ARRAY?

QuLogicQuLogic
QuLogic
  1. 
      
  2. ibis-cli/ibiscli.c (Diff revision 1)
     
     
    Show all issues

    Should this be G_OPTION_ARG_STRING_ARRAY?

    1. We could do that, but since RFC 1459 you and do JOIN #channel1,#channel2,etc we're just pasing in whatever the user gives us to the join command which means they can technically join password protected channel too if they form it right. For example, say channel 1 requires a password of abc123 if they pass --join-channels "#channel1,#channel2 abc123 they'll join both channels successfully.

  3. 
      
QuLogic
  1. Ship It!
  2. 
      
grim
Review request changed
Status:
Completed