Hi,

Code looks good and does what it says - tested on Linux with
--status-version 2 and sending SIGUSR2.

Is the first line of the commit message ("Allow querying DCO status
via management status") descriptive enough?

Since the change allows users to see DCO status without a management
interface, just by sending SIGUSR2, maybe
"multi.c: include DCO status in status version 2" would be better?

Acked-by: Lev Stipakov <lstipa...@gmail.com>


_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to