Willy, as mentioned previously, I've adapted the series to add a new `generate_unique_id()` helper function that is then used by both `stream_generate_unique_id()` and `check_generate_unique_id()`.
I have also merged the commit adding the `unique_id` field to `struct check` and the commit adding `check_generate_unique_id()` and added an explanation that is hopefully sufficiently clear to stand on its own. The "log: Stop touching `struct stream` internals for `%ID`" patch received a slight modification to also remove a now-useless `ret = NULL;`. The "Make `lf_expr` parameter of `sess_build_logline_orig()` const" patch is new and "http_fetch: Add support for checks to `unique-id` fetch" is unchanged compared to v2. Best regards Tim Duesterhus (6): CLEANUP: Make `lf_expr` parameter of `sess_build_logline_orig()` const MINOR: Add `generate_unique_id()` helper MINOR: Allow inlining of `stream_generate_unique_id()` CLEANUP: log: Stop touching `struct stream` internals for `%ID` MINOR: check: Support generating a `unique_id` for checks MINOR: http_fetch: Add support for checks to `unique-id` fetch doc/configuration.txt | 9 +++++-- include/haproxy/check-t.h | 1 + include/haproxy/check.h | 11 ++++++++ include/haproxy/log.h | 19 ++++++++++--- include/haproxy/stream.h | 2 -- reg-tests/checks/unique_id.vtc | 49 ++++++++++++++++++++++++++++++++++ src/check.c | 7 +++++ src/http_fetch.c | 17 +++++++++--- src/log.c | 44 +++++++++++++++++++++--------- src/stream.c | 26 ------------------ 10 files changed, 135 insertions(+), 50 deletions(-) create mode 100644 reg-tests/checks/unique_id.vtc -- 2.53.0
