Skip to content
This repository has been archived by the owner on Apr 24, 2023. It is now read-only.

Commit

Permalink
Merge pull request #103 from pawelpaszki/secretOutputCondFix
Browse files Browse the repository at this point in the history
added empty string check for specClient.OutputSecret
  • Loading branch information
matskiv authored Oct 16, 2019
2 parents cf1e611 + d97ee5d commit 317ffa2
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ CONSUMER_NAMESPACES=${NAMESPACE}
PROJECT=keycloak-operator
REG=quay.io
SHELL=/bin/bash
TAG=v1.9.3
TAG=v1.9.4
PKG=github.com/integr8ly/keycloak-operator
TEST_DIRS?=$(shell sh -c "find $(TOP_SRC_DIRS) -name \\*_test.go -exec dirname {} \\; | sort | uniq")
TEST_POD_NAME=keycloak-operator-test
Expand Down
2 changes: 1 addition & 1 deletion deploy/operator.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ spec:
spec:
containers:
- name: keycloak-operator
image: quay.io/integreatly/keycloak-operator:v1.9.3
image: quay.io/integreatly/keycloak-operator:v1.9.4
ports:
- containerPort: 60000
name: metrics
Expand Down
2 changes: 1 addition & 1 deletion pkg/keycloak/realm/phaseHandler.go
Original file line number Diff line number Diff line change
Expand Up @@ -507,7 +507,7 @@ func (ph *phaseHandler) reconcileClient(kcClient, specClient *v1alpha1.KeycloakC
}
}
}
if kcClient != nil && specClient != nil && specClient.OutputSecret != nil {
if kcClient != nil && specClient != nil && specClient.OutputSecret != nil && *specClient.OutputSecret != "" {
cs, err := authenticatedClient.GetClientSecret(kcClient.ID, realmName)
if err != nil {
return err
Expand Down
2 changes: 1 addition & 1 deletion version/version.go
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
package version

var (
Version = "1.9.3"
Version = "1.9.4"
)

0 comments on commit 317ffa2

Please sign in to comment.