<@U0904NTEG> Hi , Found a small bug — confirmed p...
# osctrl
f
@javuto Hi , Found a small bug — confirmed present in v0.5.4, still on
develop
too: •
LoggingKafka
is defined in
pkg/config/types.go
, and
YAMLConfigurationLogger
even has a
Kafka *KafkaLogger
field • But it's missing from the
validLogging
map in
pkg/config/validation.go
• Result: setting
logger.type: kafka
causes
osctrl-tls
to fail startup with
invalid logging method: kafka
, even though the Kafka logger is otherwise implemented • Verified directly against the
v0.5.4
tag's source — same gap there • Fix is a one-liner — just add
LoggingKafka: true
to
validLogging
Happy to open a PR for this if useful, just wanted to flag it first since I couldn't find an existing issue for it.