Stared at the code - it's "just moving code", though the part about
"where does the KS_AUTH_DEFERRED come from?" needs a bit more digging.
The new code is slightly ugly - it's the old if/else deep nesting
style, with early returns intermixed (but keeping the old else branch,
so "yes, same code" is easier to see). A followup patch might want to
refactor this to "consistent early return style".
Tested on the server side test rig (all OK), including deferred auth
by plugin and by script.
I have adjusted the "now >= ks->auth_deferred_expire" clause as
instructed and agreed on IRC, and killed a blank line.
Your patch has been applied to the master branch.
commit e55bedd4ea435f95d288e60691abc6e53b8bf014
Author: Arne Schwabe
Date: Thu May 20 17:11:44 2021 +0200
Extracting key_state deferred auth status update into function
Signed-off-by: Arne Schwabe <[email protected]>
Acked-by: Antonio Quartulli <[email protected]>
Message-Id: <[email protected]>
URL:
https://www.mail-archive.com/[email protected]/msg22420.html
Signed-off-by: Gert Doering <[email protected]>
--
kind regards,
Gert Doering
_______________________________________________
Openvpn-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openvpn-devel
