Refactor EnsureLogFilePath function in zaplogger_logpath.go to use more restrictive directory permissions

ShocOne · ShocOne · commit f5229cfbe5fd · 2024-04-18T07:40:32.000+01:00
diff --git a/logger/zaplogger_logpath.go b/logger/zaplogger_logpath.go
@@ -31,7 +31,7 @@ func EnsureLogFilePath(logPath string) (string, error) {
 
 	// Ensure the directory exists
 	dir := filepath.Dir(logPath)
-	if err := os.MkdirAll(dir, 0755); err != nil {
+	if err := os.MkdirAll(dir, 0750); err != nil {
 		return "", err
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ func EnsureLogFilePath(logPath string) (string, error) {`
`31`	`31`
`32`	`32`	`// Ensure the directory exists`
`33`	`33`	`dir := filepath.Dir(logPath)`
`34`		`- if err := os.MkdirAll(dir, 0755); err != nil {`
	`34`	`+ if err := os.MkdirAll(dir, 0750); err != nil {`
`35`	`35`	`return "", err`
`36`	`36`	`}`
`37`	`37`