Fixing license (#3878)

* Fixing sanitize license

* Fixing license

* Fixing unit tests

api/license.go

@@ -173,12 +173,18 @@ func RemoveLicense() *model.AppError {
 }
 
 func getClientLicenceConfig(c *Context, w http.ResponseWriter, r *http.Request) {
-	etag := utils.GetClientLicenseEtag()
+	etag := utils.GetClientLicenseEtag(!c.IsSystemAdmin())
 	if HandleEtag(etag, w, r) {
 		return
 	}
 
-	clientLicense := utils.ClientLicense
+	var clientLicense map[string]string
+
+	if c.IsSystemAdmin() {
+		clientLicense = utils.ClientLicense
+	} else {
+		clientLicense = utils.GetSanitizedClientLicense()
+	}
 
 	w.Header().Set(model.HEADER_ETAG_SERVER, etag)
 	w.Write([]byte(model.MapToJson(clientLicense)))

api/user.go

@@ -922,7 +922,7 @@ func getInitialLoad(c *Context, w http.ResponseWriter, r *http.Request) {
 	if c.IsSystemAdmin() {
 		il.LicenseCfg = utils.ClientLicense
 	} else {
-		il.LicenseCfg = utils.GetSantizedClientLicense()
+		il.LicenseCfg = utils.GetSanitizedClientLicense()
 	}
 
 	w.Write([]byte(il.ToJson()))

utils/license.go

@@ -141,17 +141,23 @@ func getClientLicense(l *model.License) map[string]string {
 	return props
 }
 
-func GetClientLicenseEtag() string {
+func GetClientLicenseEtag(useSanitized bool) string {
 	value := ""
 
-	for k, v := range ClientLicense {
+	lic := ClientLicense
+
+	if useSanitized {
+		lic = GetSanitizedClientLicense()
+	}
+
+	for k, v := range lic {
 		value += fmt.Sprintf("%s:%s;", k, v)
 	}
 
 	return model.Etag(fmt.Sprintf("%x", md5.Sum([]byte(value))))
 }
 
-func GetSantizedClientLicense() map[string]string {
+func GetSanitizedClientLicense() map[string]string {
 	sanitizedLicense := make(map[string]string)
 
 	for k, v := range ClientLicense {

utils/license_test.go

@@ -50,18 +50,18 @@ func TestValidateLicense(t *testing.T) {
 }
 
 func TestClientLicenseEtag(t *testing.T) {
-	etag1 := GetClientLicenseEtag()
+	etag1 := GetClientLicenseEtag(false)
 
 	ClientLicense["SomeFeature"] = "true"
 
-	etag2 := GetClientLicenseEtag()
+	etag2 := GetClientLicenseEtag(false)
 	if etag1 == etag2 {
 		t.Fatal("etags should not match")
 	}
 
 	ClientLicense["SomeFeature"] = "false"
 
-	etag3 := GetClientLicenseEtag()
+	etag3 := GetClientLicenseEtag(false)
 	if etag2 == etag3 {
 		t.Fatal("etags should not match")
 	}