Acked-by: Gert Doering
Tested on the server rig, with a client-connect script which can do deferred
("echo 2 >$client_connect_deferred_file") if requested, and will then succeed,
fail, or "disable". For "succeed" and "disable", this is all clear from the
server logs, but for "fail", you see the
4x typo
On 19/07/2020 18:34, Arne Schwabe wrote:
From: Fabian Knittel
This patch introduces the concept of a return value file for the client-connect
handlers. (This is very similar to the auth value file used during deferred
authentication.) The file name is stored in the client_connect_stat
Hi,
On Fri, Jul 17, 2020 at 02:06:02PM +0200, David Sommerseth wrote:
> On 17/07/2020 13:29, Arne Schwabe wrote:
> > The return false/return true is the result of
> > running uncrustify.
[..]
> > +case CC_RET_SKIPPED: return true;
> > +
> > +default: ASSERT(0);
>
> Code style poli
Acked-by: Gert Doering
Reviewed, change makes sense. Even better, it makes my torture test
setup (client-connect script, client-connect plugin, client-connect-v2
plugin) which can inject arbitrary "FAIL" or "disable" at any of these
points work as requested - while not breaking other tests.
T
From: Fabian Knittel
Uses the infrastructure provided and used in the previous patch to provide
deferral support to the v1 client-connect plugin handler as well.
Signed-off-by: Fabian Knittel
PATCH V3: Modify the API to also (optionally) call the plugin on a deferred
call. This allows the plug
As we never do client-connect and authentication at the same time
it is safe to reuse the existing fields for client-connect return
status file
Signed-off-by: Arne Schwabe
---
src/openvpn/multi.c | 16 +---
1 file changed, 13 insertions(+), 3 deletions(-)
diff --git a/src/openvpn/mu
This state is used to handle a corner case when multiple connect
handlers are active and one of them fails. Unfortunately, this state
complicates the state machine a bit without a good benefit.
Current behaviour:
First/all connect handler(s) fail:
- client disconnect handler is not called at a
From: Fabian Knittel
This patch introduces the concept of a return value file for the client-connect
handlers. (This is very similar to the auth value file used during deferred
authentication.) The file name is stored in the client_connect_state struct.
In addition, the patch also allows the st
The V2 API is simpler than the V1 API since there is no passing of
data via files. This also means that with the current API the V2 API
cannot support async notify via files. Adding a file just for async
notify seems very hacky and when needed we should implement a better
option when async is neede
Hi,
If your VPN establishes a route to the domain controller(s) and the
domain name resolves from the client, you can join the domain just as
you would do while directly connected to the LAN. For example, if the
domain name is example.local, "nslookup example.local" should return
the IP addresses
Acked-by: Gert Doering
Your patch has been applied to the master branch.
commit 4cabd28a77116ab3926f81258af9bdb342e0336a
Author: David Sommerseth
Date: Sun Jul 19 13:48:53 2020 +0200
doc/man: Do not install man CONTRIBUTING.rst Changes.rst files
Signed-off-by: David Sommerseth
When the man page got split up into several .rst files, these files got
listed into dist_doc_DATA=. This variable will both distribute (package
in the source tarball) and install these files into /usr/share/doc.
This was not intended, and it duplicates the content and makes the doc
dir quite messy
12 matches
Mail list logo
