Hi Alan,<br><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
1) you seem to be adding this exec patch to other places where external<br>
execution of code takes place - eg verification of the EAP cert - was that<br>
intentional?<br>
<br></blockquote><div>Yes, this is intended. I had to change radius_exec_program function which is used in various places.</div><div>The verification of the EAP cert is executed with the default timeout (10s) which is exactly the same behavior as before.</div>
<div><div>-                   (radius_exec_program(conf->make_cert_command, NULL, 1,</div><div>+                   (radius_exec_program(conf->make_cert_command, NULL, 1, RADIUS_EXEC_DEFAULT_TIMEOUT,</div></div><div>
<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
2)  it appears that you are choosing to set the value to '0' when running<br>
some commands in debug mode - surely not the intention  (or my misreading<br>
of those few lines)....<br></blockquote><div><br></div><div>The timeout value is ignored if exec_wait is set to zero.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
I agree that the timeout should be configurable.... perhaps, in an ideal world<br>
you could have a minimum and maximum. server starts with minimum value...if<br>
the reply doesnt happen quick enough and theres a failure, it could log the error<br>
and then incremement the value by one second...up until it reaches the ceiling value.<br>
errors messages would then alert admin to the issue and hopefully external systems<br>
fixed before the 'hard' error boundary reached (which is all we have right now..a hard<br>
boundary)<br><br></blockquote><div>I think it would be easier to have a timeout_warn and timeout_hard value instead of dynamically changing it.</div><div>But for now, I'd prefer to just have one timeout value.</div><div>
<br></div><div>Philipp</div><div> </div></div>