feat: drop log format 'text'

Author: tschaefer

README.md

@@ -31,17 +31,17 @@ Start the server on `localhost:8080` by default.
 ```
 For further configuration, see the command-line options below.
 
-| Flag                 | Description                                     | Default     |
-|----------------------|-------------------------------------------------|-------------|
-| `-H`, `--host`       | Host to bind the server to                      | `localhost` |
-| `-p`, `--port`       | Port to run the server on                       | `8080`      |
-| `-a`, `--auth`       | Enable bearer token authentication              | `false`     |
-| `-t`, `--token`      | Bearer token used for authentication            |             |
-| `-m`, `--metrics`    | Enable Prometheus metrics endpoint              | `false`     |
-| `-r`, `--redoc`      | Enable ReDoc API documentation                  | `false`     |
-| `-f`, `--log-format` | Set log format: `text`, `structured`, `json`    | `text`      |
-| `-l`, `--log-level`  | Set log level: `debug`, `info`, `warn`, `error` | `info`      |
-| `-h`, `--help`       | Show help for the server command                |             |
+| Flag                 | Description                                     | Default      |
+|----------------------|-------------------------------------------------|--------------|
+| `-H`, `--host`       | Host to bind the server to                      | `localhost`  |
+| `-p`, `--port`       | Port to run the server on                       | `8080`       |
+| `-a`, `--auth`       | Enable bearer token authentication              | `false`      |
+| `-t`, `--token`      | Bearer token used for authentication            |              |
+| `-m`, `--metrics`    | Enable Prometheus metrics endpoint              | `false`      |
+| `-r`, `--redoc`      | Enable ReDoc API documentation                  | `false`      |
+| `-f`, `--log-format` | Set log format: `structured`, `json`            | `structured` |
+| `-l`, `--log-level`  | Set log level: `debug`, `info`, `warn`, `error` | `info`       |
+| `-h`, `--help`       | Show help for the server command                |              |
 
 Additional a systemd service file and environment file are provided in the
 [contrib directory](https://github.com/tschaefer/rpinfo/tree/main/contrib) for automatic startup on boot and management of the

cmd/server.go

@@ -23,7 +23,7 @@ func init() {
 	serverCmd.Flags().StringP("token", "t", "", "Bearer Token for authentication")
 	serverCmd.Flags().BoolP("metrics", "m", false, "Enable Prometheus metrics")
 	serverCmd.Flags().BoolP("redoc", "r", false, "Enable ReDoc API documentation")
-	serverCmd.Flags().StringP("log-format", "f", "text", "Log format (text, structured, json)")
+	serverCmd.Flags().StringP("log-format", "f", "structured", "Log format (structured, json)")
 	serverCmd.Flags().StringP("log-level", "l", "info", "Log level (debug, info, warn, error)")
 
 	rootCmd.AddCommand(serverCmd)

server/log/log.go

@@ -36,9 +36,6 @@ func Logger(level, format string) error {
 		logger = slog.New(slog.NewTextHandler(os.Stdout, opts))
 	case "json":
 		logger = slog.New(slog.NewJSONHandler(os.Stdout, opts))
-	case "text":
-		// Use default logger, print info level only.
-		return nil
 	default:
 		return fmt.Errorf("unknown log format: %s", format)
 	}

server/log/log_test.go

@@ -30,7 +30,7 @@ func Test_LoggerReturnsErrorIfFormatIsUnknown(t *testing.T) {
 
 func Test_LoggerReturnsNoErrorIfLevelAndFormatAreKnown(t *testing.T) {
 	levels := []string{"debug", "info", "warn", "error"}
-	formats := []string{"structured", "json", "text"}
+	formats := []string{"structured", "json"}
 
 	for _, level := range levels {
 		for _, format := range formats {