<div dir="ltr"><div>I agree with Robert. <br></div><div><br></div><div>When adding those new directives, the "default" should match the old behavior so people who are not aware of the new directives will NOT get any unexpected results while upgrading to this release. People who has a need to change the old behavior now will have the new directives in hand.</div><div><br></div><div>Can you make keepalive_timeout and keepalive_requests both default to indefinite?</div><div><br></div><div>Thanks!</div><div>Frank<br></div><div><br></div><div>On Fri Aug 10 20:50:04 UTC 2018, Robert Paprocki wrote:<br>> Quite the patch. I recall this behavior being discussed a number of times<br>> in the past.<br>><br>> Question: why the default of 100? This feels like a significantly breaking<br>> change wrt. the previous behavior. Are there any plans for advanced<br>> communication regarding this change, outside of a nominal changelog entry<br>> (e.g., "introduced 'keepalive_requests' directive")?<br></div></div>