Skip to content
This repository was archived by the owner on Oct 7, 2022. It is now read-only.

Commit b31d617

Browse files
author
João Assunção
committed
fix(injector): ads error checking to pass the golangci errorchecking
1 parent fb3dc2f commit b31d617

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

cmd/injector.go

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -33,10 +33,7 @@ func main() {
3333
metrics.Register()
3434
schemaRegistry, err := schema_registry.NewSchemaRegistry(os.Getenv("SCHEMA_REGISTRY_URL"))
3535
if err != nil {
36-
err = level.Error(logger).Log("err", err, "message", "failed to create schema registry client")
37-
if err != nil {
38-
panic(err)
39-
}
36+
level.Error(logger).Log("err", err, "message", "failed to create schema registry client")
4037
}
4138

4239
kafkaConfig := &kafka.Config{

src/injector/injector.go

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,10 @@ func MakeKafkaConsumer(endpoints Endpoints, logger log.Logger, schemaRegistry *s
3939

4040
includeKey, err := strconv.ParseBool(kafkaConfig.IncludeKey)
4141
if err != nil {
42-
level.Warn(logger).Log("err", err, "message", "failed to get consumer with key and value")
42+
err = level.Warn(logger).Log("err", err, "message", "failed to get consumer include key configuration flag")
43+
if err != nil {
44+
panic(err)
45+
}
4346
includeKey = false
4447
}
4548

0 commit comments

Comments
 (0)