On Mon, May 06, 2019 at 01:29:20AM +0200, Tim Düsterhus wrote: > - What about 'resolution_pool_size'? The only thing it does is emitting > a warning (not a fatal error). I believe it can also be removed from the > documentation.
Apparently there is no single stable version with this directive, I think it was added during development and removed later. Thus both the doc entry and the parsing can be removed. > - What about 'block' and 'redispatch' and the various *timeouts which > are deprecated since more than 5 years (with HAProxy 1.5): > https://github.com/haproxy/haproxy/commit/de9d2d7b86abb0d7110bc3190aca5b26fec6fd64, > https://github.com/haproxy/haproxy/commit/a3c504c032614ee65af434e41c08ebe46855ebd8 > and > https://github.com/haproxy/haproxy/commit/ed44649eb78a8ce5efc203f273c7df774defe2af > Currently they appear to still be functional. Perhaps they can be made > non-functional (fatal error) and removed from the documentation. While I totally agree with this, I want users of 1.9 to experience the least possible friction when upgrading to 2.0 because 1.9 is not LTS. However I'm totally OK with removing them from the doc right now and removing them completely in 2.1. In addition, I wanted to remove all the req* and rsp* directives for 1.9 but I recalled it too late in the development cycle. Similarly I'd like to see all these directives emit a warning now and be removed in 2.1. Their existence causes too much trouble, they are not evaluated in the same order as the other ones, and they are totally emulated nowadays (the header block is artificially rebuilt, passed to them for regex processing, and the result is parsed again and reinjected into the headers block). Thus I would welcome a patch adding the warnings and recommendations for alternatives to all of them. This along with the legacy HTTP code removal are the things that could deserve opening the -next branch if someone is interested in starting these cleanups. BTW, I merged your two other patches. Thanks, Willy
