Hi, On Tue, Nov 29, 2022 at 4:04 PM Gert Doering <g...@greenie.muc.de> wrote:
> Hi, > > On Tue, Nov 29, 2022 at 01:37:56PM -0500, selva.n...@gmail.com wrote: > > From: Selva Nair <selva.n...@gmail.com> > > > > It seems sometimes comma-separated pulled options have > > an offending leading space. Not sure whether that is an error, > > but the change here matches the behaviour of option parsing. > > > > v2: fix typo in commit message. > > I have seen this, but I'd really prefer to fix the pushing side. > > Which options have you seen this with? I only observed with with > "auth-token", which grows an extra leading space when being sent > unsolicited. > That's the only one I have seen too -- I made this on noticing that option parser is happy with the leading space. While fixing this at the server side would be great, it's arguable that this one too is needed to bring it line with option parsing. I'm fine either way. Selva
_______________________________________________ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel
