From c13a3c3c17df33b2d64f4ec30f4b7dbb54037329 Mon Sep 17 00:00:00 2001 From: Cuong Manh Le Date: Fri, 3 Oct 2025 22:28:46 +0700 Subject: [PATCH] cmd/cli: ensure error message ends with newline --- cmd/cli/commands_service_start.go | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/cmd/cli/commands_service_start.go b/cmd/cli/commands_service_start.go index 0831371..c5430ef 100644 --- a/cmd/cli/commands_service_start.go +++ b/cmd/cli/commands_service_start.go @@ -273,26 +273,27 @@ func (sc *ServiceCommand) Start(cmd *cobra.Command, args []string) error { logger.Warn().Err(err).Msg("Failed to get logs from HTTP log server") } if len(logs) == 0 { - logger.Write([]byte(``)) + logger.Write([]byte("\n")) } else { logger.Write(logs) + logger.Write([]byte("\n")) } } else { - logger.Write([]byte(``)) + logger.Write([]byte("\n")) } } // Report any error if occurred. if err != nil { _, _ = logger.Write(marker) - msg := fmt.Sprintf("An error occurred while performing test query: %s", err) + msg := fmt.Sprintf("An error occurred while performing test query: %s\n", err) logger.Write([]byte(msg)) } // If ctrld service is running but selfCheckStatus failed, it could be related // to user's system firewall configuration, notice users about it. if status == service.StatusRunning && err == nil { _, _ = logger.Write(marker) - logger.Write([]byte(`ctrld service was running, but a DNS query could not be sent to its listener`)) - logger.Write([]byte(`Please check your system firewall if it is configured to block/intercept/redirect DNS queries`)) + logger.Write([]byte("ctrld service was running, but a DNS query could not be sent to its listener\n")) + logger.Write([]byte("Please check your system firewall if it is configured to block/intercept/redirect DNS queries\n")) } _, _ = logger.Write(marker)