[Openvpn-devel] [PATCH] Improve "recursive routing" warning

2018-05-29 Thread Lev Stipakov
- print protocol, source/dest addresses and ports - mention "--allow-recursive-routing" - add possible usecase to manpage Trac #843 Signed-off-by: Lev Stipakov --- doc/openvpn.8 | 4 ++- src/openvpn/forward.c | 89 +++ 2 files changed, 86

[Openvpn-devel] [PATCH] Improve "recursive routing" warning

2018-05-29 Thread Lev Stipakov
- print protocol, source/dest addresses and ports - mention "--allow-recursive-routing" - add possible usecase to manpage Trac #843 Signed-off-by: Lev Stipakov --- doc/openvpn.8 | 4 ++- src/openvpn/forward.c | 89 +++ 2 files changed, 86

[Openvpn-devel] [PATCH v2] Improve "recursive routing" warning

2018-05-29 Thread Lev Stipakov
V2: style fixes - print protocol, source/dest addresses and ports - mention "--allow-recursive-routing" - add possible usecase to manpage Trac #843 Signed-off-by: Lev Stipakov style fixes --- doc/openvpn.8 | 4 ++- src/openvpn/forward.c | 93 ++

Re: [Openvpn-devel] [PATCH v2] Improve "recursive routing" warning

2018-05-29 Thread Gert Doering
Hi, On Tue, May 29, 2018 at 02:47:27PM +0300, Lev Stipakov wrote: > V2: style fixes > > - print protocol, source/dest addresses and ports > - mention "--allow-recursive-routing" > - add possible usecase to manpage > > Trac #843 I'm not sure I'm convinced. This is quite a lot of extra code for

Re: [Openvpn-devel] [PATCH v2] Improve "recursive routing" warning

2018-05-29 Thread Lev Stipakov
Hi, > I'm not sure I'm convinced. This is quite a lot of extra code for > an "add a few extra fields to a warning" enhancement - especially > given that those that are already confused by the warning today > will be more confused by "more numbers"... > I assume that people are confused by the f