🐛 [Bug]: Logger Middleware: Enabling color changes padding for some fields

[JShorthouse]

The colourization code for {method} and {status} formats the value and adds extra padding either side. This padding is not added when colorization is disabled.

fiber/middleware/logger/tags.go

Lines 176 to 189 in 8ec7cec

	TagStatus: func(output Buffer, c *fiber.Ctx, data *Data, extraParam string) (int, error) {
	if cfg.enableColors {
	colors := c.App().Config().ColorScheme
	return output.WriteString(fmt.Sprintf("%s %3d %s", statusColor(c.Response().StatusCode(), colors), c.Response().StatusCode(), colors.Reset))
	}
	return appendInt(output, c.Response().StatusCode())
	},
	TagMethod: func(output Buffer, c *fiber.Ctx, data *Data, extraParam string) (int, error) {
	if cfg.enableColors {
	colors := c.App().Config().ColorScheme
	return output.WriteString(fmt.Sprintf("%s %-7s %s", methodColor(c.Method(), colors), c.Method(), colors.Reset))
	}
	return output.WriteString(c.Method())
	},

This discrepency is immediately obvious when using a custom format that puts these values in brackets, e.g:

"${time} [${locals:requestid}: ${ip}] (${status}) [${method}] ${path}​\n"

With DisableColours: false:

DisableColours: true:

[welcome]

Thanks for opening your first issue here! 🎉 Be sure to follow the issue template! If you need help or want to chat with us, join us on Discord https://gofiber.io/discord

[JShorthouse]

Personally I can live with the 7 character padding on the method type because it makes sense for "pretty printing" to ensure a constant width for all possible values.

But the single extra spaces on each side really threw me off. There's currently no way for me to remove that space and keep the colours on. I don't think having that spacing makes much sense, if people want an extra space either side they can just add it to their format string like: ( ${status} ) [ ${method} ]

[AKARSHITJOSHI]

@JShorthouse are we looking to remove the extra spacing or add extra spaces when DisableColours: false ??

[JShorthouse]

I think removing the spacing would make more sense