Merge pull request #952 from Shubhranshu153/fix-log-file-permisison

Shubhranshu153 · web-flow · commit e1f260042e47 · 2025-02-24T13:52:33.000-08:00
fix: file permissions for ecr-login
diff --git a/ecr-login/config/log.go b/ecr-login/config/log.go
@@ -39,7 +39,7 @@ func logrusConfig() {
 		fmt.Fprintf(os.Stderr, "log: failed to create directory: %v", err)
 		logdir = os.TempDir()
 	}
-	file, err := os.OpenFile(filepath.Join(logdir, "ecr-login.log"), os.O_RDWR|os.O_CREATE|os.O_APPEND, 0664)
+	file, err := os.OpenFile(filepath.Join(logdir, "ecr-login.log"), os.O_RDWR|os.O_CREATE|os.O_APPEND, 0644)
 	if err != nil {
 		return
 	}

Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ func logrusConfig() {`
`39`	`39`	`fmt.Fprintf(os.Stderr, "log: failed to create directory: %v", err)`
`40`	`40`	`logdir = os.TempDir()`
`41`	`41`	`}`
`42`		`- file, err := os.OpenFile(filepath.Join(logdir, "ecr-login.log"), os.O_RDWR\|os.O_CREATE\|os.O_APPEND, 0664)`
	`42`	`+ file, err := os.OpenFile(filepath.Join(logdir, "ecr-login.log"), os.O_RDWR\|os.O_CREATE\|os.O_APPEND, 0644)`
`43`	`43`	`if err != nil {`
`44`	`44`	`return`
`45`	`45`	`}`