Merge pull request #207 from Cryptophobia/master

fix(config.go): fix buffer path name check
vmware · Apr 21, 2021 · c0be66d · c0be66d
2 parents 39ef433 + c82dafe
commit c0be66d
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/config-reloader/config/config.go b/config-reloader/config/config.go
@@ -119,7 +119,7 @@ func (cfg *Config) Validate() error {
 		return fmt.Errorf("invalid annotation name: '%s'", cfg.AnnotConfigmapName)
 	}
 
-	if cfg.BufferMountFolder != "" && cfg.hasValidBufferMountFolder() {
+	if cfg.BufferMountFolder != "" && !cfg.hasValidBufferMountFolder() {
 		return fmt.Errorf("invalid fluentd buffer mount folder: '%s%s'", "/var/log/", cfg.BufferMountFolder)
 	}
 
@@ -288,9 +288,9 @@ func (cfg *Config) ParseFluentdLogLevel() (string, error) {
 func (cfg *Config) hasValidBufferMountFolder() bool {
 	for _, r := range cfg.BufferMountFolder {
 		if !unicode.IsLetter(r) && !unicode.IsDigit(r) && r != '-' && r != '_' {
-			continue
-		} else {
 			return false
+		} else {
+			continue
 		}
 	}
 	return true