From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-lj1-f172.google.com (mail-lj1-f172.google.com [209.85.208.172]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by dev.tarantool.org (Postfix) with ESMTPS id 39541469710 for ; Thu, 28 May 2020 13:08:28 +0300 (MSK) Received: by mail-lj1-f172.google.com with SMTP id e4so10176839ljn.4 for ; Thu, 28 May 2020 03:08:28 -0700 (PDT) From: Cyrill Gorcunov Date: Thu, 28 May 2020 13:07:34 +0300 Message-Id: <20200528100738.221911-5-gorcunov@gmail.com> In-Reply-To: <20200528100738.221911-1-gorcunov@gmail.com> References: <20200528100738.221911-1-gorcunov@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [Tarantool-patches] [PATCH v4 4/8] lua/log: do not allow to set json for boot logger List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: tml For some reason we've missed that say_set_log_format doesn't support json format not only for syslog but for boottime logging as well. Part-of #689 Signed-off-by: Cyrill Gorcunov --- src/lua/log.lua | 7 +++++-- test/app-tap/logger.test.lua | 1 - 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/src/lua/log.lua b/src/lua/log.lua index d1d712cab..2491501f7 100644 --- a/src/lua/log.lua +++ b/src/lua/log.lua @@ -160,8 +160,11 @@ local function log_format(name) end if fmt_str2num[name] == ffi.C.SF_JSON then - if ffi.C.log_type() == ffi.C.SAY_LOGGER_SYSLOG then - error("log_format: 'json' can't be used with syslog logger") + if ffi.C.log_type() == ffi.C.SAY_LOGGER_SYSLOG or + ffi.C.log_type() == ffi.C.SAY_LOGGER_BOOT then + local m = "log_format: %s can't be used with " .. + "syslog or boot-time logger" + error(m:format(fmt_num2str[ffi.C.SF_JSON])) end ffi.C.say_set_log_format(ffi.C.SF_JSON) else diff --git a/test/app-tap/logger.test.lua b/test/app-tap/logger.test.lua index 492d5ea0b..7bfa06e80 100755 --- a/test/app-tap/logger.test.lua +++ b/test/app-tap/logger.test.lua @@ -5,7 +5,6 @@ test:plan(24) -- gh-3946: Assertion failure when using log_format() before box.cfg() local log = require('log') -log.log_format('json') log.log_format('plain') -- -- 2.26.2