]> git.ipfire.org Git - thirdparty/knot-resolver.git/commitdiff
log: remove hard-coded prefix from log messages
authorLukáš Ježek <lukas.jezek@nic.cz>
Fri, 18 Jun 2021 08:14:56 +0000 (10:14 +0200)
committerTomas Krizek <tomas.krizek@nic.cz>
Thu, 29 Jul 2021 09:40:50 +0000 (11:40 +0200)
21 files changed:
daemon/bindings/cache.c
daemon/bindings/impl.h
daemon/bindings/net.c
daemon/engine.c
daemon/ffimodule.c
daemon/http.c
daemon/io.c
daemon/main.c
daemon/network.c
daemon/tls.c
daemon/tls_ephemeral_credentials.c
daemon/tls_session_ticket-srv.c
daemon/worker.c
daemon/zimport.c
lib/cache/api.c
lib/cache/cdb_lmdb.c
lib/cache/entry_list.c
lib/cache/entry_rr.c
lib/dnssec/ta.c
lib/log.c
lib/module.c

index 4f7d2f7da742a5380c65fa9b44495d6b11c3e0e7..b761fdb04c240a2273c0a32d4c18b89f52af518f 100644 (file)
@@ -206,7 +206,7 @@ static int cache_open(lua_State *L)
         * even in case of not having any work to do. */
        ret = kr_cache_check_health(&engine->resolver.cache, 5000);
        if (ret != 0) {
-               kr_log_error(LOG_GRP_CACHE, "[cache] periodic health check failed (ignored): %s\n",
+               kr_log_error(LOG_GRP_CACHE, "periodic health check failed (ignored): %s\n",
                                kr_strerror(ret));
        }
 
index 92e82a460aa9f62846feb3d550166e1b8ca81d05..0955681e43afd72f67f101d497c66b8953545081 100644 (file)
@@ -69,7 +69,7 @@ static inline int execute_callback(lua_State *L, int argc)
 {
        int ret = engine_pcall(L, argc);
        if (ret != 0) {
-               kr_log_error(LOG_GRP_SYSTEM, "[system] error: %s\n", lua_tostring(L, -1));
+               kr_log_error(LOG_GRP_SYSTEM, "error: %s\n", lua_tostring(L, -1));
        }
        /* Clear the stack, there may be event a/o enything returned */
        lua_settop(L, 0);
index b0d3d549291a6deb031eab20e4ccdd9b81b98bd9..702c84951c0e947ecc6e6e3ea1efa4f312a870b9 100644 (file)
@@ -141,7 +141,7 @@ static bool net_listen_addrs(lua_State *L, int port, endpoint_flags_t flags, int
                if (ret == 0) return true; /* success */
 
                if (is_unix) {
-                       kr_log_error(LOG_GRP_NETWORK, "[network] bind to '%s' (UNIX): %s\n",
+                       kr_log_error(LOG_GRP_NETWORK, "bind to '%s' (UNIX): %s\n",
                                        str, kr_strerror(ret));
                } else if (flags.xdp) {
                        const char *err_str = knot_strerror(ret);
@@ -157,18 +157,18 @@ static bool net_listen_addrs(lua_State *L, int port, endpoint_flags_t flags, int
                        /* Notable OK strerror: KNOT_EPERM Operation not permitted */
 
                        if (nic_queue == -1) {
-                               kr_log_error(LOG_GRP_NETWORK, "[network] failed to initialize XDP for '%s@%d'"
+                               kr_log_error(LOG_GRP_NETWORK, "failed to initialize XDP for '%s@%d'"
                                                " (nic_queue = <auto>): %s\n",
                                                str, port, err_str);
                        } else {
-                               kr_log_error(LOG_GRP_NETWORK, "[network] failed to initialize XDP for '%s@%d'"
+                               kr_log_error(LOG_GRP_NETWORK, "failed to initialize XDP for '%s@%d'"
                                                " (nic_queue = %d): %s\n",
                                                str, port, nic_queue, err_str);
                        }
 
                } else {
                        const char *stype = flags.sock_type == SOCK_DGRAM ? "UDP" : "TCP";
-                       kr_log_error(LOG_GRP_NETWORK, "[network] bind to '%s@%d' (%s): %s\n",
+                       kr_log_error(LOG_GRP_NETWORK, "bind to '%s@%d' (%s): %s\n",
                                        str, port, stype, kr_strerror(ret));
                }
                return false; /* failure */
@@ -775,7 +775,7 @@ static int net_tls_client(lua_State *L)
                ok_ca = memcmp(newcfg->pins.at[i], oldcfg->pins.at[i], TLS_SHA256_RAW_LEN) == 0;
        const bool ok_insecure = newcfg->insecure == oldcfg->insecure;
        if (!(ok_h && ok_ca && ok_pins && ok_insecure)) {
-               kr_log_warning(LOG_GRP_TLSCLIENT, "[tls_client] "
+               kr_log_warning(LOG_GRP_TLSCLIENT,
                        "warning: re-defining TLS authentication parameters for %s\n",
                        addr_str);
        }
index f0979d6a35a9c05111dbd334ec9d9f63061136c8..8a120006fb85b200d730dbe89f56cda8a4ceb0aa 100644 (file)
@@ -709,7 +709,7 @@ int engine_load_sandbox(struct engine *engine)
        /* Init environment */
        int ret = luaL_dofile(engine->L, LIBDIR "/sandbox.lua");
        if (ret != 0) {
-               fprintf(stderr, "[system] error %s\n", lua_tostring(engine->L, -1));
+               kr_log_error(LOG_GRP_SYSTEM, "error %s\n", lua_tostring(engine->L, -1));
                lua_pop(engine->L, 1);
                return kr_error(ENOEXEC);
        }
@@ -728,7 +728,7 @@ int engine_loadconf(struct engine *engine, const char *config_path)
 
        int ret = luaL_dofile(engine->L, config_path);
        if (ret != 0) {
-               fprintf(stderr, "[system] error while loading config: "
+               kr_log_error(LOG_GRP_SYSTEM, "error while loading config: "
                        "%s (workdir '%s')\n", lua_tostring(engine->L, -1), cwd);
                lua_pop(engine->L, 1);
        }
@@ -808,7 +808,7 @@ int engine_register(struct engine *engine, const char *name, const char *precede
                        ret = engine_pcall(L, 1);
                }
                if (kr_fails_assert(ret == 0)) {  /* probably not critical, but weird */
-                       kr_log_error(LOG_GRP_SYSTEM, "[system] internal error when loading C module %s: %s\n",
+                       kr_log_error(LOG_GRP_SYSTEM, "internal error when loading C module %s: %s\n",
                                        module->name, lua_tostring(L, -1));
                        lua_pop(L, 1);
                }
@@ -817,12 +817,12 @@ int engine_register(struct engine *engine, const char *name, const char *precede
                /* No luck with C module, so try to load and .init() lua module. */
                ret = ffimodule_register_lua(engine, module, name);
                if (ret != 0) {
-                       kr_log_error(LOG_GRP_SYSTEM, "[system] failed to load module '%s'\n", name);
+                       kr_log_error(LOG_GRP_SYSTEM, "failed to load module '%s'\n", name);
                }
 
        } else if (ret == kr_error(ENOTSUP)) {
                /* Print a more helpful message when module is linked against an old resolver ABI. */
-               kr_log_error(LOG_GRP_SYSTEM, "[system] module '%s' links to unsupported ABI, please rebuild it\n", name);
+               kr_log_error(LOG_GRP_SYSTEM, "module '%s' links to unsupported ABI, please rebuild it\n", name);
        }
 
        if (ret != 0) {
index 75dc183a88e8497d06df12c23ce9c1a982b14cd9..a9562f7e5a5a74f742dd23244b83ccd6da3ce4fa 100644 (file)
@@ -66,7 +66,7 @@ static int l_ffi_modcb(lua_State *L, struct kr_module *module)
        lua_pushpointer(L, module);
        if (lua_pcall(L, 2, 0, 0) == 0)
                return kr_ok();
-       kr_log_error(LOG_GRP_SYSTEM, "[system] error: %s\n", lua_tostring(L, -1));
+       kr_log_error(LOG_GRP_SYSTEM, "error: %s\n", lua_tostring(L, -1));
        lua_pop(L, 1);
        return kr_error(1);
 }
@@ -117,12 +117,12 @@ static int l_ffi_call_layer(kr_layer_t *ctx, int slot_ix)
        int state = ctx->req->state;
        if (ret) { /* Exception or another lua problem. */
                state = KR_STATE_FAIL;
-               kr_log_error(LOG_GRP_SYSTEM, "[system] error: %s\n", lua_tostring(L, -1));
+               kr_log_error(LOG_GRP_SYSTEM, "error: %s\n", lua_tostring(L, -1));
 
        } else if (lua_isnumber(L, -1)) { /* Explicitly returned state. */
                state = lua_tointeger(L, -1);
                if (!kr_state_consistent(state)) {
-                       kr_log_error(LOG_GRP_SYSTEM, "[system] error: nonsense state returned from lua module layer: %d\n",
+                       kr_log_error(LOG_GRP_SYSTEM, "error: nonsense state returned from lua module layer: %d\n",
                                        state);
                        state = KR_STATE_FAIL;
                }
@@ -136,7 +136,7 @@ static int l_ffi_call_layer(kr_layer_t *ctx, int slot_ix)
 
        } else { /* Nonsense returned. */
                state = KR_STATE_FAIL;
-               kr_log_error(LOG_GRP_SYSTEM, "[system] error: nonsense returned from lua module layer: %s\n",
+               kr_log_error(LOG_GRP_SYSTEM, "error: nonsense returned from lua module layer: %s\n",
                                lua_tostring(L, -1));
                /* Unfortunately we can't easily get name of the module/function here. */
        }
@@ -278,7 +278,7 @@ int ffimodule_register_lua(struct engine *engine, struct kr_module *module, cons
        lua_getglobal(L, "require");
        lua_pushfstring(L, "kres_modules.%s", name);
        if (lua_pcall(L, 1, LUA_MULTRET, 0) != 0) {
-               kr_log_error(LOG_GRP_SYSTEM, "[system] error: %s\n", lua_tostring(L, -1));
+               kr_log_error(LOG_GRP_SYSTEM, "error: %s\n", lua_tostring(L, -1));
                lua_pop(L, 1);
                return kr_error(ENOENT);
        }
index 944ca4e00bc69e8c2dd5bcb8d93a464578a5d6b9..b08824563fa1a64a2f9cd0cd9b2521d9484213c6 100644 (file)
@@ -409,7 +409,7 @@ static int data_chunk_recv_callback(nghttp2_session *h2, uint8_t flags, int32_t
                required += sizeof(uint16_t);
 
        if (required > remaining) {
-               kr_log_error(LOG_GRP_DOH, "[http] insufficient space in buffer\n");
+               kr_log_error(LOG_GRP_DOH, "insufficient space in buffer\n");
                ctx->incomplete_stream = -1;
                return NGHTTP2_ERR_CALLBACK_FAILURE;
        }
index 065063a2575dec86de91b0f3a4261aaa46c7f3e6..a384fef3f5cab6ab3ef55af5233925ead33a2abe 100644 (file)
@@ -165,7 +165,7 @@ int io_bind(const struct sockaddr *addr, int type, const endpoint_flags_t *flags
                int omit = IP_PMTUDISC_OMIT;
                if (type == SOCK_DGRAM && addr->sa_family == AF_INET
                    && setsockopt(fd, IPPROTO_IP, IP_MTU_DISCOVER, &omit, sizeof(omit))) {
-                       kr_log_error(LOG_GRP_IO, "[ io ]"
+                       kr_log_error(LOG_GRP_IO,
                                "failed to disable Path MTU discovery for %s UDP: %s\n",
                                kr_straddr(addr), strerror(errno));
                }
@@ -199,7 +199,7 @@ int io_listen_udp(uv_loop_t *loop, uv_udp_t *handle, int fd)
        int socklen = sizeof(union inaddr);
        ret = uv_udp_getsockname(handle, session_get_sockname(s), &socklen);
        if (ret) {
-               kr_log_error(LOG_GRP_IO, "[ io ] ERROR: getsockname failed: %s\n", uv_strerror(ret));
+               kr_log_error(LOG_GRP_IO, "ERROR: getsockname failed: %s\n", uv_strerror(ret));
                abort(); /* It might be nontrivial not to leak something here. */
        }
 
@@ -517,7 +517,7 @@ int io_listen_tcp(uv_loop_t *loop, uv_tcp_t *handle, int fd, int tcp_backlog, bo
 #if ENABLE_DOH2
                connection = https_accept;
 #else
-               kr_log_error(LOG_GRP_IO, "[ io ] kresd was compiled without libnghttp2 support\n");
+               kr_log_error(LOG_GRP_IO, "kresd was compiled without libnghttp2 support\n");
                return kr_error(ENOPROTOOPT);
 #endif
        } else if (has_tls) {
@@ -536,7 +536,7 @@ int io_listen_tcp(uv_loop_t *loop, uv_tcp_t *handle, int fd, int tcp_backlog, bo
 #ifdef TCP_DEFER_ACCEPT
        val = KR_CONN_RTT_MAX/1000;
        if (setsockopt(fd, IPPROTO_TCP, TCP_DEFER_ACCEPT, &val, sizeof(val))) {
-               kr_log_error(LOG_GRP_IO, "[ io ] listen TCP (defer_accept): %s\n", strerror(errno));
+               kr_log_error(LOG_GRP_IO, "listen TCP (defer_accept): %s\n", strerror(errno));
        }
 #endif
 
@@ -553,7 +553,7 @@ int io_listen_tcp(uv_loop_t *loop, uv_tcp_t *handle, int fd, int tcp_backlog, bo
        val = 1; /* Accepts on/off */
        #endif
        if (setsockopt(fd, IPPROTO_TCP, TCP_FASTOPEN, &val, sizeof(val))) {
-               kr_log_error(LOG_GRP_IO, "[ io ] listen TCP (fastopen): %s%s\n", strerror(errno),
+               kr_log_error(LOG_GRP_IO, "listen TCP (fastopen): %s%s\n", strerror(errno),
                        (errno != EPERM ? "" :
                         ".  This may be caused by TCP Fast Open being disabled in the OS."));
        }
@@ -686,7 +686,7 @@ void io_tty_process_input(uv_stream_t *stream, ssize_t nread, const uv_buf_t *bu
                if (data->mode == io_mode_binary) {
                        /* Leader expects length field in all cases */
                        if (!message || len_s > UINT32_MAX) {
-                               kr_log_error(LOG_GRP_IO, "[ io ] unrepresentable respose on control socket, "
+                               kr_log_error(LOG_GRP_IO, "unrepresentable respose on control socket, "
                                                "sending back empty block (command '%s')\n", cmd);
                                len_s = 0;
                        }
@@ -798,11 +798,11 @@ static void xdp_rx(uv_poll_t* handle, int status, int events)
 {
        const int XDP_RX_BATCH_SIZE = 64;
        if (status < 0) {
-               kr_log_error(LOG_GRP_XDP, "[xdp] poll status %d: %s\n", status, uv_strerror(status));
+               kr_log_error(LOG_GRP_XDP, "poll status %d: %s\n", status, uv_strerror(status));
                return;
        }
        if (events != UV_READABLE) {
-               kr_log_error(LOG_GRP_XDP, "[xdp] poll unexpected events: %d\n", events);
+               kr_log_error(LOG_GRP_XDP, "poll unexpected events: %d\n", events);
                return;
        }
 
@@ -818,10 +818,10 @@ static void xdp_rx(uv_poll_t* handle, int status, int events)
 
        if (kr_fails_assert(ret == KNOT_EOK)) {
                /* ATM other error codes can only be returned when called incorrectly */
-               kr_log_error(LOG_GRP_XDP, "[xdp] knot_xdp_recv(): %d, %s\n", ret, knot_strerror(ret));
+               kr_log_error(LOG_GRP_XDP, "knot_xdp_recv(): %d, %s\n", ret, knot_strerror(ret));
                return;
        }
-       kr_log_verbose("[xdp] poll triggered, processing a batch of %d packets\n", (int)rcvd);
+       kr_log_verbose("poll triggered, processing a batch of %d packets\n", (int)rcvd);
        kr_require(rcvd <= XDP_RX_BATCH_SIZE);
        for (int i = 0; i < rcvd; ++i) {
                const knot_xdp_msg_t *msg = &msgs[i];
@@ -850,7 +850,7 @@ static void xdp_warn_mode(const char *ifname)
 
        const unsigned if_index = if_nametoindex(ifname);
        if (!if_index) {
-               kr_log_warning(LOG_GRP_XDP, "[xdp] warning: interface %s, unexpected error when converting its name: %s\n",
+               kr_log_warning(LOG_GRP_XDP, "warning: interface %s, unexpected error when converting its name: %s\n",
                                ifname, strerror(errno));
                return;
        }
@@ -860,13 +860,13 @@ static void xdp_warn_mode(const char *ifname)
        case KNOT_XDP_MODE_FULL:
                return;
        case KNOT_XDP_MODE_EMUL:
-               kr_log_warning(LOG_GRP_XDP, "[xdp] warning: interface %s running only with XDP emulation\n",
+               kr_log_warning(LOG_GRP_XDP, "warning: interface %s running only with XDP emulation\n",
                                ifname);
                return;
        case KNOT_XDP_MODE_NONE: // enum warnings from compiler
                break;
        }
-       kr_log_warning(LOG_GRP_XDP, "[xdp] warning: interface %s running in unexpected XDP mode %d\n",
+       kr_log_warning(LOG_GRP_XDP, "warning: interface %s running in unexpected XDP mode %d\n",
                        ifname, (int)mode);
 }
 int io_listen_xdp(uv_loop_t *loop, struct endpoint *ep, const char *ifname)
index 33b7d7bb1c5d4a37a36182fabcd84088cae9cd38..1d7b3a70a77023def839aed0e9cfe09a75aa31f3 100644 (file)
@@ -149,7 +149,7 @@ static int run_worker(uv_loop_t *loop, struct engine *engine, bool leader, struc
        case UV_NAMED_PIPE:     /* echo 'quit()' | kresd ... */
                break;
        default:
-               kr_log_error(LOG_GRP_SYSTEM, "[system] "
+               kr_log_error(LOG_GRP_SYSTEM,
                        "error: standard input is not a terminal or pipe; "
                        "use '-n' if you want non-interactive mode.  "
                        "Commands can be simply added to your configuration file or sent over the control socket.\n"
@@ -349,7 +349,7 @@ static int bind_sockets(addr_array_t *addrs, bool tls, flagged_fd_array_t *fds)
                }
                free(sa);
                if (ret != 0) {
-                       kr_log_error(LOG_GRP_NETWORK, "[network] bind to '%s'%s: %s\n",
+                       kr_log_error(LOG_GRP_NETWORK, "bind to '%s'%s: %s\n",
                                addrs->at[i], tls ? " (TLS)" : "", kr_strerror(ret));
                        has_error = true;
                }
@@ -368,7 +368,7 @@ static int start_listening(struct network *net, flagged_fd_array_t *fds) {
                        /* TODO: try logging address@port.  It's not too important,
                         * because typical problems happen during binding already.
                         * (invalid address, permission denied) */
-                       kr_log_error(LOG_GRP_NETWORK, "[network] listen on fd=%d: %s\n",
+                       kr_log_error(LOG_GRP_NETWORK, "listen on fd=%d: %s\n",
                                        ffd->fd, kr_strerror(ret));
                        /* Continue printing all of these before exiting. */
                } else {
@@ -392,7 +392,7 @@ static void drop_capabilities(void)
 
                /* Apply. */
                if (capng_apply(CAPNG_SELECT_BOTH) < 0) {
-                       kr_log_error(LOG_GRP_SYSTEM, "[system] failed to set process capabilities: %s\n",
+                       kr_log_error(LOG_GRP_SYSTEM, "failed to set process capabilities: %s\n",
                                  strerror(errno));
                } else {
                        kr_log_verbose("[system] all capabilities dropped\n");
@@ -407,7 +407,7 @@ static void drop_capabilities(void)
 int main(int argc, char **argv)
 {
        if (setvbuf(stdout, NULL, _IONBF, 0) || setvbuf(stderr, NULL, _IONBF, 0)) {
-               kr_log_error(LOG_GRP_SYSTEM, "[system] failed to to set output buffering (ignored): %s\n",
+               kr_log_error(LOG_GRP_SYSTEM, "failed to to set output buffering (ignored): %s\n",
                                strerror(errno));
                fflush(stderr);
        }
@@ -432,7 +432,7 @@ int main(int argc, char **argv)
                /* FIXME: access isn't a good way if we start as root and drop privileges later */
                if (access(the_args->rundir, W_OK) != 0
                    || chdir(the_args->rundir) != 0) {
-                       kr_log_error(LOG_GRP_SYSTEM, "[system] rundir '%s': %s\n",
+                       kr_log_error(LOG_GRP_SYSTEM, "rundir '%s': %s\n",
                                        the_args->rundir, strerror(errno));
                        return EXIT_FAILURE;
                }
@@ -450,7 +450,7 @@ int main(int argc, char **argv)
                } else if (access(config, R_OK) != 0) {
                        char cwd[PATH_MAX];
                        get_workdir(cwd, sizeof(cwd));
-                       kr_log_error(LOG_GRP_SYSTEM, "[system] config '%s' (workdir '%s'): %s\n",
+                       kr_log_error(LOG_GRP_SYSTEM, "config '%s' (workdir '%s'): %s\n",
                                config, cwd, strerror(errno));
                        return EXIT_FAILURE;
                }
@@ -471,10 +471,10 @@ int main(int argc, char **argv)
                ret = setrlimit(RLIMIT_NOFILE, &rlim);
        }
        if (ret) {
-               kr_log_error(LOG_GRP_SYSTEM, "[system] failed to get or set file-descriptor limit: %s\n",
+               kr_log_error(LOG_GRP_SYSTEM, "failed to get or set file-descriptor limit: %s\n",
                                strerror(errno));
        } else if (rlim.rlim_cur < 512*1024) {
-               kr_log_warning(LOG_GRP_SYSTEM, "[system] warning: hard limit for number of file-descriptors is only %ld but recommended value is 524288\n",
+               kr_log_warning(LOG_GRP_SYSTEM, "warning: hard limit for number of file-descriptors is only %ld but recommended value is 524288\n",
                                (long)rlim.rlim_cur);
        }
 
@@ -492,13 +492,13 @@ int main(int argc, char **argv)
        static struct engine engine;
        ret = engine_init(&engine, &pool);
        if (ret != 0) {
-               kr_log_error(LOG_GRP_SYSTEM, "[system] failed to initialize engine: %s\n", kr_strerror(ret));
+               kr_log_error(LOG_GRP_SYSTEM, "failed to initialize engine: %s\n", kr_strerror(ret));
                return EXIT_FAILURE;
        }
        /* Initialize the worker */
        ret = worker_init(&engine, the_args->forks);
        if (ret != 0) {
-               kr_log_error(LOG_GRP_SYSTEM, "[system] failed to initialize worker: %s\n", kr_strerror(ret));
+               kr_log_error(LOG_GRP_SYSTEM, "failed to initialize worker: %s\n", kr_strerror(ret));
                return EXIT_FAILURE;
        }
 
@@ -525,7 +525,7 @@ int main(int argc, char **argv)
                }
        }
        if (ret) {
-               kr_log_error(LOG_GRP_SYSTEM, "[system] failed to set up signal handlers: %s\n",
+               kr_log_error(LOG_GRP_SYSTEM, "failed to set up signal handlers: %s\n",
                                strerror(abs(errno)));
                ret = EXIT_FAILURE;
                goto cleanup;
@@ -534,7 +534,7 @@ int main(int argc, char **argv)
         * (of the usual type) may skew results, e.g. epoll_pwait() taking lots of time. */
        ret = uv_loop_configure(loop, UV_LOOP_BLOCK_SIGNAL, SIGPROF);
        if (ret) {
-               kr_log_info(LOG_GRP_SYSTEM, "[system] failed to block SIGPROF in event loop, ignoring: %s\n",
+               kr_log_info(LOG_GRP_SYSTEM, "failed to block SIGPROF in event loop, ignoring: %s\n",
                                uv_strerror(ret));
        }
 
@@ -546,7 +546,7 @@ int main(int argc, char **argv)
 
        ret = udp_queue_init_global(loop);
        if (ret) {
-               kr_log_error(LOG_GRP_SYSTEM, "[system] failed to initialize UDP queue: %s\n",
+               kr_log_error(LOG_GRP_SYSTEM, "failed to initialize UDP queue: %s\n",
                                kr_strerror(ret));
                ret = EXIT_FAILURE;
                goto cleanup;
index fc627d49d507a66a4b7471a6e9243e2337bd0c34..db401daf78b1387351946c49b9b13943e16f39dd 100644 (file)
@@ -46,7 +46,7 @@ static int endpoint_open_lua_cb(struct network *net, struct endpoint *ep,
        void **pp = trie_get_try(net->endpoint_kinds, ep->flags.kind,
                                strlen(ep->flags.kind));
        if (!pp && net->missing_kind_is_error) {
-               kr_log_error(LOG_GRP_NETWORK, "[network] error: network socket kind '%s' not handled when opening '%s",
+               kr_log_error(LOG_GRP_NETWORK, "error: network socket kind '%s' not handled when opening '%s",
                                ep->flags.kind, log_addr);
                if (ep->family != AF_UNIX)
                        kr_log_error(LOG_GRP_NETWORK, "#%d", ep->port);
@@ -66,7 +66,7 @@ static int endpoint_open_lua_cb(struct network *net, struct endpoint *ep,
                lua_pushfstring(L, "%s#%d", log_addr, ep->port);
        }
        if (lua_pcall(L, 3, 0, 0)) {
-               kr_log_error(LOG_GRP_NETWORK, "[network] error opening %s: %s\n", log_addr, lua_tostring(L, -1));
+               kr_log_error(LOG_GRP_NETWORK, "error opening %s: %s\n", log_addr, lua_tostring(L, -1));
                return kr_error(ENOSYS); /* TODO: better value? */
        }
        ep->engaged = true;
@@ -106,7 +106,7 @@ static void endpoint_close_lua_cb(struct network *net, struct endpoint *ep)
        void **pp = trie_get_try(net->endpoint_kinds, ep->flags.kind,
                                strlen(ep->flags.kind));
        if (!pp && net->missing_kind_is_error) {
-               kr_log_error(LOG_GRP_NETWORK, "[network] internal error: missing kind '%s' in endpoint registry\n",
+               kr_log_error(LOG_GRP_NETWORK, "internal error: missing kind '%s' in endpoint registry\n",
                                ep->flags.kind);
                return;
        }
@@ -118,7 +118,7 @@ static void endpoint_close_lua_cb(struct network *net, struct endpoint *ep)
        lua_pushpointer(L, ep);
        lua_pushstring(L, "FIXME:endpoint-identifier");
        if (lua_pcall(L, 3, 0, 0)) {
-               kr_log_error(LOG_GRP_NETWORK, "[nework] failed to close FIXME:endpoint-identifier: %s\n",
+               kr_log_error(LOG_GRP_NETWORK, "failed to close FIXME:endpoint-identifier: %s\n",
                                lua_tostring(L, -1));
        }
 }
@@ -135,7 +135,7 @@ static void endpoint_close(struct network *net, struct endpoint *ep, bool force)
                socklen_t addr_len = sizeof(sa);
                if (getsockname(ep->fd, (struct sockaddr *)&sa, &addr_len)
                    || unlink(sa.sun_path)) {
-                       kr_log_error(LOG_GRP_NETWORK, "[network] error (ignored) when closing unix socket (fd = %d): %s\n",
+                       kr_log_error(LOG_GRP_NETWORK, "error (ignored) when closing unix socket (fd = %d): %s\n",
                                        ep->fd, strerror(errno));
                        return;
                }
@@ -292,7 +292,7 @@ static int open_endpoint(struct network *net, const char *addr_str,
        if (ep->family == AF_UNIX) {
                /* Some parts of connection handling would need more work,
                 * so let's support AF_UNIX only with .kind != NULL for now. */
-               kr_log_error(LOG_GRP_NETWORK, "[network] AF_UNIX only supported with set { kind = '...' }\n");
+               kr_log_error(LOG_GRP_NETWORK, "AF_UNIX only supported with set { kind = '...' }\n");
                ret = EAFNOSUPPORT;
                goto finish_ret;
                /*
@@ -549,9 +549,9 @@ void network_new_hostname(struct network *net, struct engine *engine)
                if (newcreds) {
                        tls_credentials_release(net->tls_credentials);
                        net->tls_credentials = newcreds;
-                       kr_log_info(LOG_GRP_TLS, "[tls] Updated ephemeral X.509 cert with new hostname\n");
+                       kr_log_info(LOG_GRP_TLS, "Updated ephemeral X.509 cert with new hostname\n");
                } else {
-                       kr_log_error(LOG_GRP_TLS, "[tls] Failed to update ephemeral X.509 cert with new hostname, using existing one\n");
+                       kr_log_error(LOG_GRP_TLS, "Failed to update ephemeral X.509 cert with new hostname, using existing one\n");
                }
        }
 }
@@ -588,7 +588,7 @@ int network_set_bpf(struct network *net, int bpf_fd)
                return 0;
        }
 #else
-       kr_log_error(LOG_GRP_NETWORK, "[network] SO_ATTACH_BPF socket option doesn't supported\n");
+       kr_log_error(LOG_GRP_NETWORK, "SO_ATTACH_BPF socket option doesn't supported\n");
        (void)net;
        (void)bpf_fd;
        return 0;
@@ -611,7 +611,7 @@ static int clear_bpf_cb(const char *key, void *val, void *ext)
                kr_require(sockfd != -1);
 
                if (setsockopt(sockfd, SOL_SOCKET, SO_DETACH_BPF, NULL, 0) != 0) {
-                       kr_log_error(LOG_GRP_NETWORK, "[network] failed to clear SO_DETACH_BPF socket option\n");
+                       kr_log_error(LOG_GRP_NETWORK, "failed to clear SO_DETACH_BPF socket option\n");
                }
                /* Proceed even if setsockopt() failed,
                 * as we want to process all opened sockets. */
@@ -625,7 +625,7 @@ void network_clear_bpf(struct network *net)
 #ifdef SO_DETACH_BPF
        map_walk(&net->endpoints, clear_bpf_cb, NULL);
 #else
-       kr_log_error(LOG_GRP_NETWORK, "[network] SO_DETACH_BPF socket option doesn't supported\n");
+       kr_log_error(LOG_GRP_NETWORK, "SO_DETACH_BPF socket option doesn't supported\n");
        (void)net;
 #endif
 }
index ef9267101810e74f3b6cf8d27c9caecb5c85b996..c3ef8b3bb6ca46510d6294854a1ffca9808c697c 100644 (file)
@@ -60,7 +60,7 @@ static int kres_gnutls_set_priority(gnutls_session_t session) {
        const char *errpos = NULL;
        int err = gnutls_priority_set_direct(session, priorities, &errpos);
        if (err != GNUTLS_E_SUCCESS) {
-               kr_log_error(LOG_GRP_TLS, "[tls] setting priority '%s' failed at character %zd (...'%s') with %s (%d)\n",
+               kr_log_error(LOG_GRP_TLS, "setting priority '%s' failed at character %zd (...'%s') with %s (%d)\n",
                             priorities, errpos - priorities, errpos, gnutls_strerror_name(err), err);
        }
        return err;
@@ -295,10 +295,10 @@ struct tls_ctx *tls_new(struct worker_ctx *worker)
        if (!net->tls_credentials) {
                net->tls_credentials = tls_get_ephemeral_credentials(worker->engine);
                if (!net->tls_credentials) {
-                       kr_log_error(LOG_GRP_TLS, "[tls] X.509 credentials are missing, and ephemeral credentials failed; no TLS\n");
+                       kr_log_error(LOG_GRP_TLS, "X.509 credentials are missing, and ephemeral credentials failed; no TLS\n");
                        return NULL;
                }
-               kr_log_info(LOG_GRP_TLS, "[tls] Using ephemeral TLS credentials\n");
+               kr_log_info(LOG_GRP_TLS, "Using ephemeral TLS credentials\n");
                tls_credentials_log_pins(net->tls_credentials);
        }
 
@@ -311,15 +311,15 @@ struct tls_ctx *tls_new(struct worker_ctx *worker)
                                if (newcreds) {
                                        tls_credentials_release(net->tls_credentials);
                                        net->tls_credentials = newcreds;
-                                       kr_log_info(LOG_GRP_TLS, "[tls] Renewed expiring ephemeral X.509 cert\n");
+                                       kr_log_info(LOG_GRP_TLS, "Renewed expiring ephemeral X.509 cert\n");
                                } else {
-                                       kr_log_error(LOG_GRP_TLS, "[tls] Failed to renew expiring ephemeral X.509 cert, using existing one\n");
+                                       kr_log_error(LOG_GRP_TLS, "Failed to renew expiring ephemeral X.509 cert, using existing one\n");
                                }
                        }
                } else {
                        /* non-ephemeral cert: warn once when certificate expires */
                        if (now >= net->tls_credentials->valid_until) {
-                               kr_log_error(LOG_GRP_TLS, "[tls] X.509 certificate has expired!\n");
+                               kr_log_error(LOG_GRP_TLS, "X.509 certificate has expired!\n");
                                net->tls_credentials->valid_until = GNUTLS_X509_NO_WELL_DEFINED_EXPIRATION;
                        }
                }
@@ -327,13 +327,13 @@ struct tls_ctx *tls_new(struct worker_ctx *worker)
 
        struct tls_ctx *tls = calloc(1, sizeof(struct tls_ctx));
        if (tls == NULL) {
-               kr_log_error(LOG_GRP_TLS, "[tls] failed to allocate TLS context\n");
+               kr_log_error(LOG_GRP_TLS, "failed to allocate TLS context\n");
                return NULL;
        }
 
        int err = gnutls_init(&tls->c.tls_session, GNUTLS_SERVER | GNUTLS_NONBLOCK);
        if (err != GNUTLS_E_SUCCESS) {
-               kr_log_error(LOG_GRP_TLS, "[tls] gnutls_init(): %s (%d)\n", gnutls_strerror_name(err), err);
+               kr_log_error(LOG_GRP_TLS, "gnutls_init(): %s (%d)\n", gnutls_strerror_name(err), err);
                tls_free(tls);
                return NULL;
        }
@@ -341,7 +341,7 @@ struct tls_ctx *tls_new(struct worker_ctx *worker)
        err = gnutls_credentials_set(tls->c.tls_session, GNUTLS_CRD_CERTIFICATE,
                                     tls->credentials->credentials);
        if (err != GNUTLS_E_SUCCESS) {
-               kr_log_error(LOG_GRP_TLS, "[tls] gnutls_credentials_set(): %s (%d)\n", gnutls_strerror_name(err), err);
+               kr_log_error(LOG_GRP_TLS, "gnutls_credentials_set(): %s (%d)\n", gnutls_strerror_name(err), err);
                tls_free(tls);
                return NULL;
        }
@@ -605,7 +605,7 @@ void tls_credentials_log_pins(struct tls_credentials *tls_credentials)
                                                        index, &certs, &cert_count);
                if (err != GNUTLS_E_SUCCESS) {
                        if (err != GNUTLS_E_REQUESTED_DATA_NOT_AVAILABLE) {
-                               kr_log_error(LOG_GRP_TLS, "[tls] could not get X.509 certificates (%d) %s\n",
+                               kr_log_error(LOG_GRP_TLS, "could not get X.509 certificates (%d) %s\n",
                                                err, gnutls_strerror_name(err));
                        }
                        return;
@@ -615,10 +615,10 @@ void tls_credentials_log_pins(struct tls_credentials *tls_credentials)
                        char pin[TLS_SHA256_BASE64_BUFLEN] = { 0 };
                        err = get_oob_key_pin(certs[i], pin, sizeof(pin), false);
                        if (err != GNUTLS_E_SUCCESS) {
-                               kr_log_error(LOG_GRP_TLS, "[tls] could not calculate RFC 7858 OOB key-pin from cert %d (%d) %s\n",
+                               kr_log_error(LOG_GRP_TLS, "could not calculate RFC 7858 OOB key-pin from cert %d (%d) %s\n",
                                                i, err, gnutls_strerror_name(err));
                        } else {
-                               kr_log_info(LOG_GRP_TLS, "[tls] RFC 7858 OOB key-pin (%d): pin-sha256=\"%s\"\n",
+                               kr_log_info(LOG_GRP_TLS, "RFC 7858 OOB key-pin (%d): pin-sha256=\"%s\"\n",
                                                i, pin);
                        }
                        gnutls_x509_crt_deinit(certs[i]);
@@ -653,17 +653,17 @@ static time_t _get_end_entity_expiration(gnutls_certificate_credentials_t creds)
        time_t ret = GNUTLS_X509_NO_WELL_DEFINED_EXPIRATION;
 
        if ((err = gnutls_certificate_get_crt_raw(creds, 0, 0, &data)) != GNUTLS_E_SUCCESS) {
-               kr_log_error(LOG_GRP_TLS, "[tls] failed to get cert to check expiration: (%d) %s\n",
+               kr_log_error(LOG_GRP_TLS, "failed to get cert to check expiration: (%d) %s\n",
                             err, gnutls_strerror_name(err));
                goto done;
        }
        if ((err = gnutls_x509_crt_init(&cert)) != GNUTLS_E_SUCCESS) {
-               kr_log_error(LOG_GRP_TLS, "[tls] failed to initialize cert: (%d) %s\n",
+               kr_log_error(LOG_GRP_TLS, "failed to initialize cert: (%d) %s\n",
                             err, gnutls_strerror_name(err));
                goto done;
        }
        if ((err = gnutls_x509_crt_import(cert, &data, GNUTLS_X509_FMT_DER)) != GNUTLS_E_SUCCESS) {
-               kr_log_error(LOG_GRP_TLS, "[tls] failed to construct cert while checking expiration: (%d) %s\n",
+               kr_log_error(LOG_GRP_TLS, "failed to construct cert while checking expiration: (%d) %s\n",
                             err, gnutls_strerror_name(err));
                goto done;
        }
@@ -689,14 +689,14 @@ int tls_certificate_set(struct network *net, const char *tls_cert, const char *t
 
        int err = 0;
        if ((err = gnutls_certificate_allocate_credentials(&tls_credentials->credentials)) != GNUTLS_E_SUCCESS) {
-               kr_log_error(LOG_GRP_TLS, "[tls] gnutls_certificate_allocate_credentials() failed: (%d) %s\n",
+               kr_log_error(LOG_GRP_TLS, "gnutls_certificate_allocate_credentials() failed: (%d) %s\n",
                             err, gnutls_strerror_name(err));
                tls_credentials_free(tls_credentials);
                return kr_error(ENOMEM);
        }
        if ((err = gnutls_certificate_set_x509_system_trust(tls_credentials->credentials)) < 0) {
                if (err != GNUTLS_E_UNIMPLEMENTED_FEATURE) {
-                       kr_log_warning(LOG_GRP_TLS, "[tls] warning: gnutls_certificate_set_x509_system_trust() failed: (%d) %s\n",
+                       kr_log_warning(LOG_GRP_TLS, "warning: gnutls_certificate_set_x509_system_trust() failed: (%d) %s\n",
                                     err, gnutls_strerror_name(err));
                        tls_credentials_free(tls_credentials);
                        return err;
@@ -712,7 +712,7 @@ int tls_certificate_set(struct network *net, const char *tls_cert, const char *t
        if ((err = gnutls_certificate_set_x509_key_file(tls_credentials->credentials,
                                                        tls_cert, tls_key, GNUTLS_X509_FMT_PEM)) != GNUTLS_E_SUCCESS) {
                tls_credentials_free(tls_credentials);
-               kr_log_error(LOG_GRP_TLS, "[tls] gnutls_certificate_set_x509_key_file(%s,%s) failed: %d (%s)\n",
+               kr_log_error(LOG_GRP_TLS, "gnutls_certificate_set_x509_key_file(%s,%s) failed: %d (%s)\n",
                             tls_cert, tls_key, err, gnutls_strerror_name(err));
                return kr_error(EINVAL);
        }
@@ -829,7 +829,7 @@ tls_client_param_t * tls_client_param_new()
        e->refs = 1;
        int ret = gnutls_certificate_allocate_credentials(&e->credentials);
        if (ret != GNUTLS_E_SUCCESS) {
-               kr_log_error(LOG_GRP_TLSCLIENT, "[tls_client] error: gnutls_certificate_allocate_credentials() fails (%s)\n",
+               kr_log_error(LOG_GRP_TLSCLIENT, "error: gnutls_certificate_allocate_credentials() fails (%s)\n",
                             gnutls_strerror_name(ret));
                free(e);
                return NULL;
@@ -961,12 +961,12 @@ static int client_verify_pin(const unsigned int cert_list_size,
                                params->pins.len);
        }
 
-       kr_log_error(LOG_GRP_TLSCLIENT, "[tls_client] no pin matched: %zu pins * %d certificates\n",
+       kr_log_error(LOG_GRP_TLSCLIENT, "no pin matched: %zu pins * %d certificates\n",
                        params->pins.len, cert_list_size);
        return GNUTLS_E_CERTIFICATE_ERROR;
 
 #else /* TLS_CAN_USE_PINS */
-       kr_log_error(LOG_GRP_TLSCLIENT, "[tls_client] internal inconsistency: TLS_CAN_USE_PINS\n");
+       kr_log_error(LOG_GRP_TLSCLIENT, "internal inconsistency: TLS_CAN_USE_PINS\n");
        kr_assert(false);
        return GNUTLS_E_CERTIFICATE_ERROR;
 #endif
@@ -981,7 +981,7 @@ static int client_verify_pin(const unsigned int cert_list_size,
 static int client_verify_certchain(gnutls_session_t tls_session, const char *hostname)
 {
        if (kr_fails_assert(hostname)) {
-               kr_log_error(LOG_GRP_TLSCLIENT, "[tls_client] internal config inconsistency: no hostname set\n");
+               kr_log_error(LOG_GRP_TLSCLIENT, "internal config inconsistency: no hostname set\n");
                return GNUTLS_E_CERTIFICATE_ERROR;
        }
 
@@ -996,16 +996,16 @@ static int client_verify_certchain(gnutls_session_t tls_session, const char *hos
                ret = gnutls_certificate_verification_status_print(
                        status, gnutls_certificate_type_get(tls_session), &msg, 0);
                if (ret == GNUTLS_E_SUCCESS) {
-                       kr_log_error(LOG_GRP_TLSCLIENT, "[tls_client] failed to verify peer certificate: "
+                       kr_log_error(LOG_GRP_TLSCLIENT, "failed to verify peer certificate: "
                                        "%s\n", msg.data);
                        gnutls_free(msg.data);
                } else {
-                       kr_log_error(LOG_GRP_TLSCLIENT, "[tls_client] failed to verify peer certificate: "
+                       kr_log_error(LOG_GRP_TLSCLIENT, "failed to verify peer certificate: "
                                        "unable to print reason: %s (%s)\n",
                                        gnutls_strerror(ret), gnutls_strerror_name(ret));
                } /* gnutls_certificate_verification_status_print end */
        } else {
-               kr_log_error(LOG_GRP_TLSCLIENT, "[tls_client] failed to verify peer certificate: "
+               kr_log_error(LOG_GRP_TLSCLIENT, "failed to verify peer certificate: "
                             "gnutls_certificate_verify_peers3 error: %s (%s)\n",
                             gnutls_strerror(ret), gnutls_strerror_name(ret));
        } /* gnutls_certificate_verify_peers3 end */
@@ -1029,7 +1029,7 @@ static int client_verify_certificate(gnutls_session_t tls_session)
 
        gnutls_certificate_type_t cert_type = gnutls_certificate_type_get(tls_session);
        if (cert_type != GNUTLS_CRT_X509) {
-               kr_log_error(LOG_GRP_TLSCLIENT, "[tls_client] invalid certificate type %i has been received\n",
+               kr_log_error(LOG_GRP_TLSCLIENT, "invalid certificate type %i has been received\n",
                             cert_type);
                return GNUTLS_E_CERTIFICATE_ERROR;
        }
@@ -1037,7 +1037,7 @@ static int client_verify_certificate(gnutls_session_t tls_session)
        const gnutls_datum_t *cert_list =
                gnutls_certificate_get_peers(tls_session, &cert_list_size);
        if (cert_list == NULL || cert_list_size == 0) {
-               kr_log_error(LOG_GRP_TLSCLIENT, "[tls_client] empty certificate list\n");
+               kr_log_error(LOG_GRP_TLSCLIENT, "empty certificate list\n");
                return GNUTLS_E_CERTIFICATE_ERROR;
        }
 
index 584a892c4c2ac2b9f81eef911b44adf9fc519b08..2cab8b4b2a4d72f94ae1f91d646d04c2e5f8cd16 100644 (file)
@@ -63,12 +63,12 @@ static gnutls_x509_privkey_t get_ephemeral_privkey ()
         * with a shared cache don't both create the same privkey: */
        lock = _lock_filename(EPHEMERAL_PRIVKEY_FILENAME ".lock");
        if (_lock_is_invalid(lock)) {
-               kr_log_error(LOG_GRP_TLS, "[tls] unable to lock lockfile " EPHEMERAL_PRIVKEY_FILENAME ".lock\n");
+               kr_log_error(LOG_GRP_TLS, "unable to lock lockfile " EPHEMERAL_PRIVKEY_FILENAME ".lock\n");
                goto done;
        }
        
        if ((err = gnutls_x509_privkey_init (&privkey)) < 0) {
-               kr_log_error(LOG_GRP_TLS, "[tls] gnutls_x509_privkey_init() failed: %d (%s)\n",
+               kr_log_error(LOG_GRP_TLS, "gnutls_x509_privkey_init() failed: %d (%s)\n",
                             err, gnutls_strerror_name(err));
                goto done;
        }
@@ -81,22 +81,22 @@ static gnutls_x509_privkey_t get_ephemeral_privkey ()
                struct stat stat;
                ssize_t bytes_read;
                if (fstat(datafd, &stat)) {
-                       kr_log_error(LOG_GRP_TLS, "[tls] unable to stat ephemeral private key " EPHEMERAL_PRIVKEY_FILENAME "\n");
+                       kr_log_error(LOG_GRP_TLS, "unable to stat ephemeral private key " EPHEMERAL_PRIVKEY_FILENAME "\n");
                        goto bad_data;
                }
                data.data = gnutls_malloc(stat.st_size);
                if (data.data == NULL) {
-                       kr_log_error(LOG_GRP_TLS, "[tls] unable to allocate memory for reading ephemeral private key\n");
+                       kr_log_error(LOG_GRP_TLS, "unable to allocate memory for reading ephemeral private key\n");
                        goto bad_data;
                }
                data.size = stat.st_size;
                bytes_read = read(datafd, data.data, stat.st_size);
                if (bytes_read != stat.st_size) {
-                       kr_log_error(LOG_GRP_TLS, "[tls] unable to read ephemeral private key\n");
+                       kr_log_error(LOG_GRP_TLS, "unable to read ephemeral private key\n");
                        goto bad_data;
                }
                if ((err = gnutls_x509_privkey_import (privkey, &data, GNUTLS_X509_FMT_PEM)) < 0) {
-                       kr_log_error(LOG_GRP_TLS, "[tls] gnutls_x509_privkey_import() failed: %d (%s)\n",
+                       kr_log_error(LOG_GRP_TLS, "gnutls_x509_privkey_import() failed: %d (%s)\n",
                                     err, gnutls_strerror_name(err));
                        /* goto bad_data; */
                bad_data:
@@ -115,25 +115,25 @@ static gnutls_x509_privkey_t get_ephemeral_privkey ()
 #else
                if ((err = gnutls_x509_privkey_generate(privkey, GNUTLS_PK_RSA, gnutls_sec_param_to_pk_bits(GNUTLS_PK_RSA, GNUTLS_SEC_PARAM_MEDIUM), 0)) < 0) {
 #endif
-                       kr_log_error(LOG_GRP_TLS, "[tls] gnutls_x509_privkey_init() failed: %d (%s)\n",
+                       kr_log_error(LOG_GRP_TLS, "gnutls_x509_privkey_init() failed: %d (%s)\n",
                                     err, gnutls_strerror_name(err));
                        gnutls_x509_privkey_deinit(privkey);
                        goto done;
                }
                /* ... and save */
-               kr_log_info(LOG_GRP_TLS, "[tls] Stashing ephemeral private key in " EPHEMERAL_PRIVKEY_FILENAME "\n");
+               kr_log_info(LOG_GRP_TLS, "Stashing ephemeral private key in " EPHEMERAL_PRIVKEY_FILENAME "\n");
                if ((err = gnutls_x509_privkey_export2(privkey, GNUTLS_X509_FMT_PEM, &data)) < 0) {
-                       kr_log_error(LOG_GRP_TLS, "[tls] gnutls_x509_privkey_export2() failed: %d (%s), not storing\n",
+                       kr_log_error(LOG_GRP_TLS, "gnutls_x509_privkey_export2() failed: %d (%s), not storing\n",
                                     err, gnutls_strerror_name(err));
                } else {
                        datafd = open(EPHEMERAL_PRIVKEY_FILENAME, O_WRONLY|O_CREAT, 0600);
                        if (datafd == -1) {
-                               kr_log_error(LOG_GRP_TLS, "[tls] failed to open " EPHEMERAL_PRIVKEY_FILENAME " to store the ephemeral key\n");
+                               kr_log_error(LOG_GRP_TLS, "failed to open " EPHEMERAL_PRIVKEY_FILENAME " to store the ephemeral key\n");
                        } else {
                                ssize_t bytes_written;
                                bytes_written = write(datafd, data.data, data.size);
                                if (bytes_written != data.size)
-                                       kr_log_error(LOG_GRP_TLS, "[tls] failed to write %d octets to "
+                                       kr_log_error(LOG_GRP_TLS, "failed to write %d octets to "
                                                     EPHEMERAL_PRIVKEY_FILENAME
                                                     " (%zd written)\n",
                                                     data.size, bytes_written);
@@ -164,7 +164,7 @@ static gnutls_x509_crt_t get_ephemeral_cert(gnutls_x509_privkey_t privkey, const
 
 #define gtx(fn, ...)                                                   \
        if ((err = fn ( __VA_ARGS__ )) != GNUTLS_E_SUCCESS) {           \
-               kr_log_error(LOG_GRP_TLS, "[tls] " #fn "() failed: %d (%s)\n",  \
+               kr_log_error(LOG_GRP_TLS, #fn "() failed: %d (%s)\n",   \
                             err, gnutls_strerror_name(err));           \
                goto bad; }
 
@@ -199,20 +199,20 @@ struct tls_credentials * tls_get_ephemeral_credentials(struct engine *engine)
 
        creds = calloc(1, sizeof(*creds));
        if (!creds) {
-               kr_log_error(LOG_GRP_TLS, "[tls] failed to allocate memory for ephemeral credentials\n");
+               kr_log_error(LOG_GRP_TLS, "failed to allocate memory for ephemeral credentials\n");
                return NULL;
        }
        if ((err = gnutls_certificate_allocate_credentials(&(creds->credentials))) < 0) {
-               kr_log_error(LOG_GRP_TLS, "[tls] failed to allocate memory for ephemeral credentials\n");
+               kr_log_error(LOG_GRP_TLS, "failed to allocate memory for ephemeral credentials\n");
                goto failure;
        }
 
        creds->valid_until = now + EPHEMERAL_CERT_EXPIRATION_SECONDS;
        creds->ephemeral_servicename = strdup(engine_get_hostname(engine));
        if (creds->ephemeral_servicename == NULL) {
-               kr_log_error(LOG_GRP_TLS, "[tls] could not get server's hostname, using '" INVALID_HOSTNAME "' instead\n");
+               kr_log_error(LOG_GRP_TLS, "could not get server's hostname, using '" INVALID_HOSTNAME "' instead\n");
                if ((creds->ephemeral_servicename = strdup(INVALID_HOSTNAME)) == NULL) {
-                       kr_log_error(LOG_GRP_TLS, "[tls] failed to allocate memory for ephemeral credentials\n");
+                       kr_log_error(LOG_GRP_TLS, "failed to allocate memory for ephemeral credentials\n");
                        goto failure;
                }
        }               
@@ -223,7 +223,7 @@ struct tls_credentials * tls_get_ephemeral_credentials(struct engine *engine)
                goto failure;
        }
        if ((err = gnutls_certificate_set_x509_key(creds->credentials, &cert, 1, privkey)) < 0) {
-               kr_log_error(LOG_GRP_TLS, "[tls] failed to set up ephemeral credentials\n");
+               kr_log_error(LOG_GRP_TLS, "failed to set up ephemeral credentials\n");
                goto failure;
        }
        gnutls_x509_privkey_deinit(privkey);
index bc30707e49d04c8dee62f7ccbc478c356537f4e7..cf8f6eb2727b611daa98208a2210e65e73d60e6c 100644 (file)
@@ -80,7 +80,7 @@ static tst_ctx_t * tst_key_create(const char *secret, size_t secret_len, uv_loop
                return NULL;
        #if !TLS_SESSION_RESUMPTION_SYNC
                if (secret_len) {
-                       kr_log_error(LOG_GRP_TLS, "[tls] session ticket: secrets were not enabled at compile-time (your GnuTLS version is not supported)\n");
+                       kr_log_error(LOG_GRP_TLS, "session ticket: secrets were not enabled at compile-time (your GnuTLS version is not supported)\n");
                        return NULL; /* ENOTSUP */
                }
        #endif
@@ -166,7 +166,7 @@ static void tst_key_check(uv_timer_t *timer, bool force_update)
        /* Compute the current epoch. */
        struct timeval now;
        if (gettimeofday(&now, NULL)) {
-               kr_log_error(LOG_GRP_TLS, "[tls] session ticket: gettimeofday failed, %s\n",
+               kr_log_error(LOG_GRP_TLS, "session ticket: gettimeofday failed, %s\n",
                                strerror(errno));
                return;
        }
@@ -177,7 +177,7 @@ static void tst_key_check(uv_timer_t *timer, bool force_update)
         * for gnutls_session_ticket_enable_server() doesn't say. */
        int err = tst_key_update(stst, epoch, force_update);
        if (err) {
-               kr_log_error(LOG_GRP_TLS, "[tls] session ticket: failed rotation, %s\n",
+               kr_log_error(LOG_GRP_TLS, "session ticket: failed rotation, %s\n",
                                kr_strerror(err));
                if (kr_fails_assert(err != kr_error(EINVAL)))
                        return;
@@ -195,7 +195,7 @@ static void tst_key_check(uv_timer_t *timer, bool force_update)
                        (uint64_t)epoch, remain_ms);
        err = uv_timer_start(timer, &tst_timer_callback, remain_ms, 0);
        if (kr_fails_assert(err == 0)) {
-               kr_log_error(LOG_GRP_TLS, "[tls] session ticket: failed to schedule, %s\n",
+               kr_log_error(LOG_GRP_TLS, "session ticket: failed to schedule, %s\n",
                                uv_strerror(err));
                return;
        }
@@ -213,7 +213,7 @@ void tls_session_ticket_enable(struct tls_session_ticket_ctx *ctx, gnutls_sessio
        };
        int err = gnutls_session_ticket_enable_server(session, &gd);
        if (err) {
-               kr_log_error(LOG_GRP_TLS, "[tls] failed to enable session tickets: %s (%d)\n",
+               kr_log_error(LOG_GRP_TLS, "failed to enable session tickets: %s (%d)\n",
                                gnutls_strerror_name(err), err);
                /* but continue without tickets */
        }
index 06b48452cfa3c54693f6e8467db428ac10944e31..cfeee9e2a75568ff01c5390f57a57bdb31c91de6 100644 (file)
@@ -1276,7 +1276,7 @@ static void xdp_tx_waker(uv_idle_t *handle)
 {
        int ret = knot_xdp_send_finish(handle->data);
        if (ret != KNOT_EAGAIN && ret != KNOT_EOK)
-               kr_log_error(LOG_GRP_XDP, "[xdp] check: ret = %d, %s\n", ret, knot_strerror(ret));
+               kr_log_error(LOG_GRP_XDP, "check: ret = %d, %s\n", ret, knot_strerror(ret));
        /* Apparently some drivers need many explicit wake-up calls
         * even if we push no additional packets (in case they accumulated a lot) */
        if (ret != KNOT_EAGAIN)
index de05e88d9daa734ac87db2332986f9c359cc8525..0f352fd26dd38973f2b32c8c5f152bf6ba005f5e 100644 (file)
@@ -465,14 +465,14 @@ static void zi_zone_process(uv_timer_t* handle)
        KR_DNAME_GET_STR(zone_name_str, z_import->origin);
 
        if (strcmp(".", zone_name_str) != 0) {
-               kr_log_error(LOG_GRP_ZIMPORT, "[zimport] unexpected zone name `%s` (root zone expected), fail\n",
+               kr_log_error(LOG_GRP_ZIMPORT, "unexpected zone name `%s` (root zone expected), fail\n",
                             zone_name_str);
                failed = 1;
                goto finish;
        }
 
        if (z_import->rrset_sorted.len <= 0) {
-               kr_log_error(LOG_GRP_ZIMPORT, "[zimport] zone `%s` is empty\n", zone_name_str);
+               kr_log_error(LOG_GRP_ZIMPORT, "zone `%s` is empty\n", zone_name_str);
                goto finish;
        }
 
@@ -490,7 +490,7 @@ static void zi_zone_process(uv_timer_t* handle)
        knot_rrset_t *rr_key = map_get(&z_import->rrset_indexed, key);
        if (!rr_key) {
                /* DNSKEY MUST be here. If not found - fail. */
-               kr_log_error(LOG_GRP_ZIMPORT, "[zimport] DNSKEY not found for `%s`, fail\n", zone_name_str);
+               kr_log_error(LOG_GRP_ZIMPORT, "DNSKEY not found for `%s`, fail\n", zone_name_str);
                failed = 1;
                goto finish;
        }
@@ -499,7 +499,7 @@ static void zi_zone_process(uv_timer_t* handle)
        map_t *trust_anchors = &z_import->worker->engine->resolver.trust_anchors;
        knot_rrset_t *rr_ta = kr_ta_get(trust_anchors, z_import->origin);
        if (!rr_ta) {
-               kr_log_error(LOG_GRP_ZIMPORT, "[zimport] error: TA for zone `%s` vanished, fail", zone_name_str);
+               kr_log_error(LOG_GRP_ZIMPORT, "error: TA for zone `%s` vanished, fail", zone_name_str);
                failed = 1;
                goto finish;
        }
@@ -596,7 +596,7 @@ finish:
        if (failed != 0) {
                if (ns_imported == 0 && other_imported == 0) {
                        import_state = -1;
-                       kr_log_error(LOG_GRP_ZIMPORT, "[zimport] import failed; zone `%s` \n", zone_name_str);
+                       kr_log_error(LOG_GRP_ZIMPORT, "import failed; zone `%s` \n", zone_name_str);
                } else {
                        import_state = 1;
                }
@@ -615,13 +615,13 @@ static int zi_record_store(zs_scanner_t *s)
 {
        if (s->r_data_length > UINT16_MAX) {
                /* Due to knot_rrset_add_rdata(..., const uint16_t size, ...); */
-               kr_log_error(LOG_GRP_ZSCANNER, "[zscanner] line %"PRIu64": rdata is too long\n",
+               kr_log_error(LOG_GRP_ZSCANNER, "line %"PRIu64": rdata is too long\n",
                                s->line_counter);
                return -1;
        }
 
        if (knot_dname_size(s->r_owner) != strlen((const char *)(s->r_owner)) + 1) {
-               kr_log_error(LOG_GRP_ZSCANNER, "[zscanner] line %"PRIu64
+               kr_log_error(LOG_GRP_ZSCANNER, "line %"PRIu64
                                ": owner name contains zero byte, skip\n",
                                s->line_counter);
                return 0;
@@ -632,14 +632,14 @@ static int zi_record_store(zs_scanner_t *s)
        knot_rrset_t *new_rr = knot_rrset_new(s->r_owner, s->r_type, s->r_class,
                                              s->r_ttl, &z_import->pool);
        if (!new_rr) {
-               kr_log_error(LOG_GRP_ZSCANNER, "[zscanner] line %"PRIu64": error creating rrset\n",
+               kr_log_error(LOG_GRP_ZSCANNER, "line %"PRIu64": error creating rrset\n",
                                s->line_counter);
                return -1;
        }
        int res = knot_rrset_add_rdata(new_rr, s->r_data, s->r_data_length,
                                       &z_import->pool);
        if (res != KNOT_EOK) {
-               kr_log_error(LOG_GRP_ZSCANNER, "[zscanner] line %"PRIu64": error adding rdata to rrset\n",
+               kr_log_error(LOG_GRP_ZSCANNER, "line %"PRIu64": error adding rdata to rrset\n",
                                s->line_counter);
                return -1;
        }
@@ -653,7 +653,7 @@ static int zi_record_store(zs_scanner_t *s)
        res = kr_rrkey(key, new_rr->rclass, new_rr->owner, new_rr->type,
                       additional_key_field);
        if (res <= 0) {
-               kr_log_error(LOG_GRP_ZSCANNER, "[zscanner] line %"PRIu64": error constructing rrkey\n",
+               kr_log_error(LOG_GRP_ZSCANNER, "line %"PRIu64": error constructing rrkey\n",
                                s->line_counter);
                return -1;
        }
@@ -666,7 +666,7 @@ static int zi_record_store(zs_scanner_t *s)
                res = map_set(&z_import->rrset_indexed, key, new_rr);
        }
        if (res != 0) {
-               kr_log_error(LOG_GRP_ZSCANNER, "[zscanner] line %"PRIu64": error saving parsed rrset\n",
+               kr_log_error(LOG_GRP_ZSCANNER, "line %"PRIu64": error saving parsed rrset\n",
                                s->line_counter);
                return -1;
        }
@@ -692,29 +692,29 @@ static int zi_state_parsing(zs_scanner_t *s)
                        }
                        break;
                case ZS_STATE_ERROR:
-                       kr_log_error(LOG_GRP_ZSCANNER, "[zscanner] line: %"PRIu64
+                       kr_log_error(LOG_GRP_ZSCANNER, "line: %"PRIu64
                                     ": parse error; code: %i ('%s')\n",
                                     s->line_counter, s->error.code,
                                     zs_strerror(s->error.code));
                        return -1;
                case ZS_STATE_INCLUDE:
-                       kr_log_error(LOG_GRP_ZSCANNER, "[zscanner] line: %"PRIu64
+                       kr_log_error(LOG_GRP_ZSCANNER, "line: %"PRIu64
                                     ": INCLUDE is not supported\n",
                                     s->line_counter);
                        return -1;
                case ZS_STATE_EOF:
                case ZS_STATE_STOP:
                        if (empty) {
-                               kr_log_error(LOG_GRP_ZIMPORT, "[zimport] empty zone file\n");
+                               kr_log_error(LOG_GRP_ZIMPORT, "empty zone file\n");
                                return -1;
                        }
                        if (!((zone_import_ctx_t *) s->process.data)->origin) {
-                               kr_log_error(LOG_GRP_ZIMPORT, "[zimport] zone file doesn't contain SOA record\n");
+                               kr_log_error(LOG_GRP_ZIMPORT, "zone file doesn't contain SOA record\n");
                                return -1;
                        }
                        return (s->error.counter == 0) ? 0 : -1;
                default:
-                       kr_log_error(LOG_GRP_ZSCANNER, "[zimport] line: %"PRIu64
+                       kr_log_error(LOG_GRP_ZSCANNER, "line: %"PRIu64
                                     ": unexpected parse state: %i\n",
                                     s->line_counter, s->state);
                        return -1;
@@ -733,7 +733,7 @@ int zi_zone_import(struct zone_import_ctx *z_import,
 
        zs_scanner_t *s = malloc(sizeof(zs_scanner_t));
        if (s == NULL) {
-               kr_log_error(LOG_GRP_ZSCANNER, "[zscanner] error creating instance of zone scanner (malloc() fails)\n");
+               kr_log_error(LOG_GRP_ZSCANNER, "error creating instance of zone scanner (malloc() fails)\n");
                return -1;
        }
 
@@ -741,7 +741,7 @@ int zi_zone_import(struct zone_import_ctx *z_import,
         * so don't print error code as it meaningless. */
        int res = zs_init(s, origin, rclass, ttl);
        if (res != 0) {
-               kr_log_error(LOG_GRP_ZSCANNER, "[zscanner] error initializing zone scanner instance, error: %i (%s)\n",
+               kr_log_error(LOG_GRP_ZSCANNER, "error initializing zone scanner instance, error: %i (%s)\n",
                             s->error.code, zs_strerror(s->error.code));
                free(s);
                return -1;
@@ -749,7 +749,7 @@ int zi_zone_import(struct zone_import_ctx *z_import,
 
        res = zs_set_input_file(s, zone_file);
        if (res != 0) {
-               kr_log_error(LOG_GRP_ZSCANNER, "[zscanner] error opening zone file `%s`, error: %i (%s)\n",
+               kr_log_error(LOG_GRP_ZSCANNER, "error opening zone file `%s`, error: %i (%s)\n",
                             zone_file, s->error.code, zs_strerror(s->error.code));
                zs_deinit(s);
                free(s);
@@ -760,7 +760,7 @@ int zi_zone_import(struct zone_import_ctx *z_import,
         * Parsing as well error processing will be performed in zi_state_parsing().
         * Store pointer to zone import context for further use. */
        if (zs_set_processing(s, NULL, NULL, (void *)z_import) != 0) {
-               kr_log_error(LOG_GRP_ZSCANNER, "[zscanner] zs_set_processing() failed for zone file `%s`, "
+               kr_log_error(LOG_GRP_ZSCANNER, "zs_set_processing() failed for zone file `%s`, "
                                "error: %i (%s)\n",
                                zone_file, s->error.code, zs_strerror(s->error.code));
                zs_deinit(s);
@@ -784,7 +784,7 @@ int zi_zone_import(struct zone_import_ctx *z_import,
                                /* For now - fail.
                                 * TODO - query DS and continue after answer had been obtained. */
                                KR_DNAME_GET_STR(zone_name_str, z_import->origin);
-                               kr_log_error(LOG_GRP_ZIMPORT, "[zimport] no TA found for `%s`, fail\n", zone_name_str);
+                               kr_log_error(LOG_GRP_ZIMPORT, "no TA found for `%s`, fail\n", zone_name_str);
                                ret = 1;
                        }
                        elapsed = kr_now() - z_import->start_timestamp;
@@ -795,7 +795,7 @@ int zi_zone_import(struct zone_import_ctx *z_import,
        free(s);
 
        if (ret != 0) {
-               kr_log_error(LOG_GRP_ZSCANNER, "[zscanner] error parsing zone file `%s`\n", zone_file);
+               kr_log_error(LOG_GRP_ZSCANNER, "error parsing zone file `%s`\n", zone_file);
                z_import->started = false;
                return ret;
        }
index ad7606f894f5e802e86400049f37a5c150ccce62..583c61211ff968c1593f302d93476971c3ae4eef 100644 (file)
@@ -81,7 +81,7 @@ static int assert_right_version(struct kr_cache *cache)
                 * Recreate cache and write version key. */
                ret = cache_op(cache, count);
                if (ret != 0) { /* Log for non-empty cache to limit noise on fresh start. */
-                       kr_log_info(LOG_GRP_CACHE, "[cache] incompatible cache database detected, purging\n");
+                       kr_log_info(LOG_GRP_CACHE, "incompatible cache database detected, purging\n");
                        if (oldret) {
                                kr_log_verbose("[cache] reading version returned: %d\n", oldret);
                        } else if (val.len != sizeof(CACHE_VERSION)) {
@@ -143,7 +143,7 @@ int kr_cache_open(struct kr_cache *cache, const struct kr_cdb_api *api, struct k
        if (ret == 0 && opts->maxsize) {
                size_t maxsize = cache->api->get_maxsize(cache->db);
                if (maxsize > opts->maxsize) kr_log_warning(LOG_GRP_CACHE,
-                       "[cache] Warning: real cache size is %zu instead of the requested %zu bytes."
+                       "Warning: real cache size is %zu instead of the requested %zu bytes."
                        "  To reduce the size you need to remove the file '%s' by hand.\n",
                        maxsize, opts->maxsize, fpath);
        }
@@ -690,7 +690,7 @@ static int stash_rrarray_entry(ranked_rr_array_t *arr, int arr_i,
        ssize_t written = stash_rrset(cache, qry, rr, rr_sigs, qry->timestamp.tv_sec,
                                        entry->rank, nsec_pmap, needs_pkt);
        if (written < 0) {
-               kr_log_error(LOG_GRP_CACHE, "[%05u.%02u][cach] stash failed, ret = %d\n", qry->request->uid,
+               kr_log_error(LOG_GRP_CACHE, "[%05u.%02u] stash failed, ret = %d\n", qry->request->uid,
                             qry->uid, ret);
                return (int) written;
        }
index b9a4f62a862f696418c070e60c097dcf3ae10d7a..3d93ded533ced4918f61ef3ef66ed57d4e3a5287 100644 (file)
@@ -87,7 +87,7 @@ static int lmdb_error(int error)
        case MDB_TXN_FULL:
                return kr_error(ENOSPC);
        default:
-               kr_log_error(LOG_GRP_CACHE, "[cache] LMDB error: %s\n", mdb_strerror(error));
+               kr_log_error(LOG_GRP_CACHE, "LMDB error: %s\n", mdb_strerror(error));
                return kr_error(error);
        }
 }
@@ -116,7 +116,7 @@ static int refresh_mapsize(struct lmdb_env *env)
 
        env->mapsize = info.me_mapsize;
        if (env->mapsize != env->st_size) {
-               kr_log_info(LOG_GRP_CACHE, "[cache] suspicious size of cache file '%s'"
+               kr_log_info(LOG_GRP_CACHE, "suspicious size of cache file '%s'"
                                ": file size %zu != LMDB map size %zu\n",
                                env->mdb_data_path, (size_t)env->st_size, env->mapsize);
        }
@@ -128,10 +128,10 @@ static void clear_stale_readers(struct lmdb_env *env)
        int cleared;
        int ret = mdb_reader_check(env->env, &cleared);
        if (ret != MDB_SUCCESS) {
-               kr_log_error(LOG_GRP_CACHE, "[cache] failed to clear stale reader locks: "
+               kr_log_error(LOG_GRP_CACHE, "failed to clear stale reader locks: "
                                "LMDB error %d %s\n", ret, mdb_strerror(ret));
        } else if (cleared != 0) {
-               kr_log_info(LOG_GRP_CACHE, "[cache] cleared %d stale reader locks\n", cleared);
+               kr_log_info(LOG_GRP_CACHE, "cleared %d stale reader locks\n", cleared);
        }
 }
 
@@ -160,7 +160,7 @@ retry:
        }
 
        if (unlikely(ret == MDB_MAP_RESIZED)) {
-               kr_log_info(LOG_GRP_CACHE, "[cache] detected size increased by another process\n");
+               kr_log_info(LOG_GRP_CACHE, "detected size increased by another process\n");
                ret = refresh_mapsize(env);
                if (ret == 0)
                        goto retry;
@@ -384,7 +384,7 @@ static int cdb_open_env(struct lmdb_env *env, const char *path, const size_t map
        if (ret == EINVAL) {
                /* POSIX says this can happen when the feature isn't supported by the FS.
                 * We haven't seen this happen on Linux+glibc but it was reported on FreeBSD.*/
-               kr_log_info(LOG_GRP_CACHE, "[cache] space pre-allocation failed and ignored; "
+               kr_log_info(LOG_GRP_CACHE, "space pre-allocation failed and ignored; "
                                "your (file)system probably doesn't support it.\n");
        } else if (ret != 0) {
                mdb_txn_abort(txn);
@@ -495,7 +495,7 @@ static int cdb_check_health(kr_cdb_pt db, struct kr_cdb_stats *stats)
         * contrary to methods based on mdb_env_info(). */
        if (st.st_size == env->st_size)
                return kr_ok();
-       kr_log_info(LOG_GRP_CACHE, "[cache] detected size change (by another instance?) of file '%s'"
+       kr_log_info(LOG_GRP_CACHE, "detected size change (by another instance?) of file '%s'"
                        ": file size %zu -> file size %zu\n",
                        env->mdb_data_path, (size_t)env->st_size, (size_t)st.st_size);
        env->st_size = st.st_size; // avoid retrying in cycle even if we fail
@@ -558,7 +558,7 @@ static int cdb_clear(kr_cdb_pt db, struct kr_cdb_stats *stats)
                                return ret;
                        }
                }
-               kr_log_info(LOG_GRP_CACHE, "[cache] clearing error, falling back\n");
+               kr_log_info(LOG_GRP_CACHE, "clearing error, falling back\n");
        }
        /* Fallback: we'll remove the database files and reopen.
         * Other instances can continue to use the removed lmdb,
@@ -582,7 +582,7 @@ static int cdb_clear(kr_cdb_pt db, struct kr_cdb_stats *stats)
        /* Find if we get a lock on lockfile. */
        const int lockfile_fd = lockfile_get(lockfile);
        if (lockfile_fd < 0) {
-               kr_log_error(LOG_GRP_CACHE, "[cache] clearing failed to get ./krcachelock (%s); retry later\n",
+               kr_log_error(LOG_GRP_CACHE, "clearing failed to get ./krcachelock (%s); retry later\n",
                                kr_strerror(lockfile_fd));
                /* As we're out of space (almost certainly - mdb_drop didn't work),
                 * we will retry on the next failing write operation. */
@@ -608,7 +608,7 @@ static int cdb_clear(kr_cdb_pt db, struct kr_cdb_stats *stats)
        /* Environment updated, release lockfile. */
        int lrerr = lockfile_release(lockfile_fd);
        if (lrerr) {
-               kr_log_error(LOG_GRP_CACHE, "[cache] failed to release ./krcachelock: %s\n",
+               kr_log_error(LOG_GRP_CACHE, "failed to release ./krcachelock: %s\n",
                                kr_strerror(lrerr));
        }
        return ret;
index fb6c457c1e5c06d9246ca718d0b160f47b7eae6a..5276d5a5a29b4f03cfb8e97068a40472cbf2679d 100644 (file)
@@ -185,7 +185,7 @@ static int cache_write_or_clear(struct kr_cache *cache, const knot_db_val_t *key
         */
        const uint64_t now = kr_now();
        if (!ignoring_errors_until) { // First error after a success.
-               kr_log_info(LOG_GRP_CACHE, "[cache] LMDB refusing writes (ignored for 5-9s): %s\n",
+               kr_log_info(LOG_GRP_CACHE, "LMDB refusing writes (ignored for 5-9s): %s\n",
                                kr_strerror(ret));
                ignoring_errors_until = now + 5000 + kr_rand_bytes(2)/16;
                return kr_error(ret);
@@ -202,7 +202,7 @@ recovery: // Try to recover by clearing cache.
                                kr_strerror(ret));
                abort();
        case 0:
-               kr_log_info(LOG_GRP_CACHE, "[cache] stuck cache cleared\n");
+               kr_log_info(LOG_GRP_CACHE, "stuck cache cleared\n");
                ignoring_errors_until = 0;
        case -EAGAIN: // fall-through; krcachelock race -> retry later
                return kr_error(ENOSPC);
index 0699726406a6d2b393c6c19b0176ab55fd78d326..a16eff5a9e6c0ec298c97e44764af22901ad977f 100644 (file)
@@ -98,7 +98,7 @@ int entry2answer(struct answer *ans, int id,
                /* Sanity check: we consumed exactly all data. */
                int unused_bytes = eh_bound - (uint8_t *)eh->data - data_off - ret;
                if (kr_fails_assert(unused_bytes == 0)) {
-                       kr_log_error(LOG_GRP_CACHE, "[cach] entry2answer ERROR: unused bytes: %d\n",
+                       kr_log_error(LOG_GRP_CACHE, "entry2answer ERROR: unused bytes: %d\n",
                                        unused_bytes);
                        ret = kr_error(EILSEQ);
                        goto fail; /* to be on the safe side */
index bfcfc8e0eebb574fa5abb452ddeb61149ea8e0d9..27591d37c3bb14d2c211cc43f1818ce7f11be653 100644 (file)
@@ -58,14 +58,14 @@ static int dnskey2ds(dnssec_binary_t *dst, const knot_dname_t *owner, const uint
        const bool flags_ok = kr_dnssec_key_zsk(rdata) && !kr_dnssec_key_revoked(rdata);
        if (!flags_ok) {
                auto_free char *owner_str = kr_dname_text(owner);
-               kr_log_error(LOG_GRP_TA, "[ ta ] refusing to trust %s DNSKEY because of flags %d\n",
+               kr_log_error(LOG_GRP_TA, "refusing to trust %s DNSKEY because of flags %d\n",
                        owner_str, dnssec_key_get_flags(key));
                ret = kr_error(EILSEQ);
                goto cleanup;
        } else if (!kr_dnssec_key_ksk(rdata)) {
                auto_free char *owner_str = kr_dname_text(owner);
                int flags = dnssec_key_get_flags(key);
-               kr_log_warning(LOG_GRP_TA, "[ ta ] warning: %s DNSKEY is missing the SEP bit; "
+               kr_log_warning(LOG_GRP_TA, "warning: %s DNSKEY is missing the SEP bit; "
                        "flags %d instead of %d\n",
                        owner_str, flags, flags + 1/*a little ugly*/);
        }
index 5feb42b699881bb55712149a7ed4f118e6f65fd9..694447daee8844c58f55936a6ff78db0d61e5a6a 100644 (file)
--- a/lib/log.c
+++ b/lib/log.c
@@ -104,7 +104,7 @@ void kr_log_fmt(log_groups_t group, log_level_t level, const char *file,
 
 static void kres_gnutls_log(int level, const char *message)
 {
-       kr_log_debug(LOG_GRP_GNUTLS, "[gnutls] (%d) %s", level, message);
+       kr_log_debug(LOG_GRP_GNUTLS, "(%d) %s", level, message);
 }
 
 char *kr_log_level2name(log_level_t level)
index e351d9f5e63e1000d04e5212449722b91d960105..e33091e47ee8dcd8f6c17578560f72b7e9c32351 100644 (file)
@@ -87,7 +87,7 @@ static int load_sym_c(struct kr_module *module, uint32_t api_required)
            || load_symbol(module->lib, m_prefix, "props")) {
                /* In case someone re-compiled against new kresd
                 * but haven't actually changed the symbols. */
-               kr_log_error(LOG_GRP_SYSTEM, "[system] module %s requires upgrade.  Please refer to "
+               kr_log_error(LOG_GRP_SYSTEM, "module %s requires upgrade.  Please refer to "
                        "https://knot-resolver.readthedocs.io/en/stable/upgrading.html",
                        module->name);
                return kr_error(ENOTSUP);