Merge pull request #390 from nan-yu/master

k8s-ci-robot · web-flow · commit 581139e56c4b · 2021-05-26T14:49:38.000-07:00
Grant read access for the error file to all users
diff --git a/cmd/git-sync/main.go b/cmd/git-sync/main.go
@@ -259,6 +259,9 @@ func (l *customLogger) writeContent(content []byte) {
 		l.Logger.Error(err, "can't rename to error-file", "temp-file", tmpFile.Name(), "error-file", errorFile)
 		return
 	}
+	if err := os.Chmod(errorFile, 0644) ; err != nil {
+		l.Logger.Error(err, "can't change the mod of the error-file", "error-file", errorFile)
+	}
 }
 
 // deleteErrorFile deletes the error file.

Original file line number	Diff line number	Diff line change
`@@ -259,6 +259,9 @@ func (l *customLogger) writeContent(content []byte) {`
`259`	`259`	`l.Logger.Error(err, "can't rename to error-file", "temp-file", tmpFile.Name(), "error-file", errorFile)`
`260`	`260`	`return`
`261`	`261`	`}`
	`262`	`+ if err := os.Chmod(errorFile, 0644) ; err != nil {`
	`263`	`+ l.Logger.Error(err, "can't change the mod of the error-file", "error-file", errorFile)`
	`264`	`+ }`
`262`	`265`	`}`
`263`	`266`
`264`	`267`	`// deleteErrorFile deletes the error file.`