From 43c538eab66fed681ee719ccc76caf4c99cb066c Mon Sep 17 00:00:00 2001 From: Willy Tarreau Date: Wed, 5 Sep 2018 14:58:15 +0200 Subject: [PATCH] MINOR: log: move the log code to sess_build_logline() to add extra arguments The current build_logline() can only be used with valid streams, which means it is not suitable for use from muxes. We start by moving it into another more generic function which takes the session as an argument, to avoid complexifying all the internal API for jsut a few use cases. This new function is not supposed to be called directly from outside so we'll be able to instrument it to support several calling conventions. For now the behaviour and conditions remain unchanged. --- include/proto/log.h | 10 +++++++++- src/log.c | 5 ++--- 2 files changed, 11 insertions(+), 4 deletions(-) diff --git a/include/proto/log.h b/include/proto/log.h index 05a7acc55..3e17733ce 100644 --- a/include/proto/log.h +++ b/include/proto/log.h @@ -34,6 +34,8 @@ #include #include +#include + extern struct pool_head *pool_head_requri; extern struct pool_head *pool_head_uniqueid; @@ -63,7 +65,13 @@ void deinit_log_buffers(); /* * Builds a log line. */ -int build_logline(struct stream *s, char *dst, size_t maxsize, struct list *list_format); +int sess_build_logline(struct session *sess, struct stream *s, char *dst, size_t maxsize, struct list *list_format); + +static inline int build_logline(struct stream *s, char *dst, size_t maxsize, struct list *list_format) +{ + return sess_build_logline(strm_sess(s), s, dst, maxsize, list_format); +} + /* * send a log for the stream when we have enough info about it. diff --git a/src/log.c b/src/log.c index dcb2ba069..4e7818da0 100644 --- a/src/log.c +++ b/src/log.c @@ -1566,11 +1566,10 @@ void deinit_log_buffers() /* Builds a log line in based on , and stops before reaching * characters. Returns the size of the output string in characters, * not counting the trailing zero which is always added if the resulting size - * is not zero. + * is not zero. It requires a session and a stream. */ -int build_logline(struct stream *s, char *dst, size_t maxsize, struct list *list_format) +int sess_build_logline(struct session *sess, struct stream *s, char *dst, size_t maxsize, struct list *list_format) { - struct session *sess = strm_sess(s); struct proxy *fe = sess->fe; struct proxy *be = s->be; struct http_txn *txn = s->txn;