vpnutil: Update 'usage' to mention 'list' option

[Tormen]

vpnutil: Update 'usage' to mention 'list' option

[Timac]

Hey @Tormen ,

The vpnutil command line tool already supports a list parameter. When running the command vpnutil list, you will see an output like:

VPN1 Connected
VPN2 Disconnected
VPN3 Disconnected

where VPN1, VPN2 and VPN3 are the names of the VPNs.

I assume that you would prefer to get a JSON output format like:

{
  "VPN1": {
    "status": "Connected"
  },
  "VPN2": {
    "status": "Disconnected"
  },
  "VPN3": {
    "status": "Disconnected"
  }
}

Would this JSON work for you? Is there anything missing in this output that you would like to see?

[Tormen]

Hi @Timac ,

Thank you!

In this case I would suggest the following to avoid similar confusion for others:
Could you please change the output when calling "vpnutil" to show that there is such a list parameter ?

As for me the "Usage" did NOT talk about "list" I (had to assume) that there is no list parameter :D

This is what I see when executing vpnutil without parameters (latest stable precompiled version, downloaded today from this GitHub):

Usage: vpnutil [start|stop] [VPN name]
Examples:
	 To start the VPN called 'MyVPN':
	 vpnutil start MyVPN

	 To stop the VPN called 'MyVPN':
	 vpnutil stop MyVPN

	 To list all available VPNs and their state:
	 vpnutil list

	 To get the status of the VPN called 'MyVPN':
	 vpnutil status MyVPN

Copyright © 2018-2023 Alexandre Colucci
blog.timac.org

[Tormen]

Hi @Timac ,

I had another look at the current output of the list command and:

The output of the list command could be improved
by using a "TAB" character, instead of "SPACE"
to separate the NAME of the VPN and the status,
as the VPN NAME /can/ also contain spaces.

Tormen

[Timac]

@Tormen Thanks again for the great suggestions. I implemented 2 changes:

• The usage now mentions the list parameter
• vpnutil list now outputs a json format. This issue was also tracked by #12 . If you run vpnutil list, you should now see a json like this:

{
  "VPNs" : [
    {
      "name" : "VPN1",
      "status" : "Connected"
    },
    {
      "name" : "VPN2",
      "status" : "Disconnected"
    },
    {
      "name" : "VPN3",
      "status" : "Disconnected"
    }
  ]
}