Hi, On Mon, Oct 08, 2018 at 08:16:16PM +0200, Arne Schwabe wrote: > There is no user facing way to enable this feature and way that feature > works (username build from MAC of primary net device) is questionable. > > It also does not compile anymore.
Feature-ACK, but the patch itself puzzles me.
> @@ -455,51 +455,6 @@ get_auth_challenge(const char *auth_challenge, struct
> gc_arena *gc)
>
> #endif /* ifdef ENABLE_MANAGEMENT */
>
> -#if AUTO_USERID
[..]
> diff --git a/src/openvpn/ssl.c b/src/openvpn/ssl.c
> index 0a947c6e..5a136d69 100644
> --- a/src/openvpn/ssl.c
> +++ b/src/openvpn/ssl.c
> @@ -410,9 +410,6 @@ auth_user_pass_setup(const char *auth_file, const struct
> static_challenge_info *
> auth_user_pass_enabled = true;
> if (!auth_user_pass.defined && !auth_token.defined)
> {
> -#if AUTO_USERID
> - get_user_pass_auto_userid(&auth_user_pass, auth_file);
> -#else
> #ifdef ENABLE_MANAGEMENT
These hunks have "ENABLE_MANAGEMENT" in the context, while all my branches
(2.3, 2.4, master) have "ENABLE_CLIENT_CR" here.
This is not really needed to remove the (questionable) feature - and I can
just adjust the patch on the fly - but I wonder where these are coming
from in your tree, and whether I missed (or messed up) something...
gert
--
"If was one thing all people took for granted, was conviction that if you
feed honest figures into a computer, honest figures come out. Never doubted
it myself till I met a computer with a sense of humor."
Robert A. Heinlein, The Moon is a Harsh Mistress
Gert Doering - Munich, Germany [email protected]
signature.asc
Description: PGP signature
_______________________________________________ Openvpn-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openvpn-devel
