> 
> 
>>      multi->tas_last = now;
> 
> I think keeping this line here is wrong.
> 
> Now that the logic has been changed, the code is setting tas_last to now
> every time this function is called (it wasn't the case before this patch).
> 
> For this reason "cached" will always be true and the ACF will never be
> checked.
> 
> A fix would be to add a condition above this line, like:
> 
> if (!cached)
>     multi->tas_last = now;
> 
> Does it make sense?
> In my tests this fixes the problem and deferred auth works again.

Yes. It is. I am not sure how I missed that.

Arne


_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to