Merge pull request #13 from jpoizat/master

Add Sha256 sum instead of full configmap in convertConfigMapToToken
fabric8io · Jan 14, 2019 · 3619145 · 3619145
2 parents 9c02368 + 063ee1d
commit 3619145
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/controller/controller.go b/controller/controller.go
@@ -19,6 +19,8 @@ import (
 	"k8s.io/kubernetes/pkg/runtime"
 	"k8s.io/kubernetes/pkg/watch"
 
+	"crypto/sha256"
+	"encoding/base64"
 	"sort"
 
 	oclient "github.com/openshift/origin/pkg/client"
@@ -224,9 +226,10 @@ func convertConfigMapToToken(cm *api.ConfigMap) string {
 	}
 	sort.Strings(values)
 	text := strings.Join(values, ";")
-	// we could zip and base64 encode
-	// but for now we could leave this easy to read so that its easier to diagnose when & why things changed
-	return text
+	hasher := sha256.New()
+	hasher.Write([]byte(text))
+	sha := base64.URLEncoding.EncodeToString(hasher.Sum(nil))
+	return sha
 }
 
 func updateContainers(containers []api.Container, annotationValue, configMapVersion string) bool {

diff --git a/version/VERSION b/version/VERSION
@@ -1 +1 @@
-2.3.11
+2.3.12