diff --git a/internal/ingress/controller/parser/kongstate/consumer.go b/internal/ingress/controller/parser/kongstate/consumer.go index f762252..12ee3a0 100644 --- a/internal/ingress/controller/parser/kongstate/consumer.go +++ b/internal/ingress/controller/parser/kongstate/consumer.go @@ -40,7 +40,7 @@ func (c *Consumer) SetCredential(log logrus.FieldLogger, credType string, credCo // See https://github.com/Kong/deck/pull/223 and https://github.com/Kong/kubernetes-ingress-controller/issues/532 // for more discussion. if cred.Key == nil { - return fmt.Errorf("key-auth for consumer %s is invalid: no key", *c.Username) + return fmt.Errorf("key-auth for consumer is invalid: no key") } c.KeyAuths = append(c.KeyAuths, &cred) case "basic-auth", "basicauth_credential": diff --git a/internal/ingress/controller/parser/kongstate/consumer_test.go b/internal/ingress/controller/parser/kongstate/consumer_test.go index 88b5a6c..08d9380 100644 --- a/internal/ingress/controller/parser/kongstate/consumer_test.go +++ b/internal/ingress/controller/parser/kongstate/consumer_test.go @@ -46,6 +46,16 @@ func TestConsumer_SetCredential(t *testing.T) { }, wantErr: false, }, + { + name: "key-auth without key", + args: args{ + credType: "key-auth", + consumer: &Consumer{}, + credConfig: map[string]string{}, + }, + result: &Consumer{}, + wantErr: true, + }, { name: "keyauth_credential", args: args{