Re: [Openvpn-devel] [PATCH] Do not include auth-token in pulled option digest

2022-12-19 Thread Gert Doering
Hi, On Mon, Dec 19, 2022 at 01:21:37PM +0100, Frank Lichtenheld wrote: > > +/* peer-id and auth-token might change on restart and this should > > not trigger reopening tun */ > > +if (strprefix(line, "peer-id ") || strprefix(line, "auth-token")) > > If I interpret this correctly,

Re: [Openvpn-devel] [PATCH] Do not include auth-token in pulled option digest

2022-12-19 Thread Frank Lichtenheld
On Sun, Dec 18, 2022 at 02:22:03PM -0500, selva.n...@gmail.com wrote: > From: Selva Nair > > As change in auth-token is common on restart and does not > require tun-reopen, exclude it from the "pulled options digest" > calculation. Without this tun is always re-opened on SIGUSR1 > if auth-token i

Re: [Openvpn-devel] [PATCH] Do not include auth-token in pulled option digest

2022-12-19 Thread Arne Schwabe
Am 18.12.22 um 20:22 schrieb selva.n...@gmail.com: From: Selva Nair As change in auth-token is common on restart and does not require tun-reopen, exclude it from the "pulled options digest" calculation. Without this tun is always re-opened on SIGUSR1 if auth-token is in use which breaks persist