Hi Arne, Thanks for reviewing this :)
On 12-11-17 21:09, Arne Schwabe wrote: > Am 11.11.17 um 17:11 schrieb Gert van Dijk: >> With NCP turned off, this will still display the cipher used. >> >> Trac: #814 >> >> Signed-off-by: Gert van Dijk <g...@gertvandijk.net> >> --- >> doc/openvpn.8 | 2 +- >> src/openvpn/multi.c | 11 +++++++---- >> 2 files changed, 8 insertions(+), 5 deletions(-) >> >> diff --git a/doc/openvpn.8 b/doc/openvpn.8 >> index 267497fd..00dbff6f 100644 >> --- a/doc/openvpn.8 >> +++ b/doc/openvpn.8 >> @@ -2477,7 +2477,7 @@ Connected Since. >> .B 2 >> \-\- a more reliable format for external processing. Compared to version 1, >> the >> client list contains some additional fields: Virtual Address, Virtual IPv6 >> -Address, Username, Client ID, Peer ID. >> +Address, Username, Client ID, Peer ID, Data Channel Cipher. >> Future versions may extend the number of fields. >> .br >> .B 3 > > > This part of the patch is okay but does not apply cleanly for me. Might > need some on the fly fixing on apply. > Did you apply the first part ([PATCH 1/2] manpage: improve description of --status and --status-version)? >> @@ -940,8 +942,8 @@ multi_print_status(struct multi_context *m, struct >> status_output *so, const int >> */ >> status_printf(so, "TITLE%c%s", sep, title_string); >> status_printf(so, "TIME%c%s%c%u", sep, time_string(now, 0, >> false, &gc_top), sep, (unsigned int)now); >> - status_printf(so, "HEADER%cCLIENT_LIST%cCommon Name%cReal >> Address%cVirtual Address%cVirtual IPv6 Address%cBytes Received%cBytes >> Sent%cConnected Since%cConnected Since (time_t)%cUsername%cClient ID%cPeer >> ID", >> - sep, sep, sep, sep, sep, sep, sep, sep, sep, sep, >> sep, sep); >> + status_printf(so, "HEADER%cCLIENT_LIST%cCommon Name%cReal >> Address%cVirtual Address%cVirtual IPv6 Address%cBytes Received%cBytes >> Sent%cConnected Since%cConnected Since (time_t)%cUsername%cClient ID%cPeer >> ID%cData Channel Cipher", > > > > Extremely long line, but not your fault, so I will let it pass. > > > Works for me and code is good so: > > Acked-by: Arne Schwabe <a...@rfc2549.org> > Tested-by: Arne Schwabe <a...@rfc2549.org>> > > Arne > ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot _______________________________________________ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel
