Commit 9572fdcf authored by GuyCheung's avatar GuyCheung

update error on type of w.Perm; change unit test perm value

parent cb3f240f
......@@ -161,7 +161,7 @@ func (w *fileLogWriter) createLogFile() (*os.File, error) {
fd, err := os.OpenFile(w.Filename, os.O_WRONLY|os.O_APPEND|os.O_CREATE, os.FileMode(perm))
if err == nil {
// Make sure file perm is user set perm cause of `os.OpenFile` will obey umask
os.Chmod(w.Filename, w.Perm)
os.Chmod(w.Filename, os.FileMode(perm))
}
return fd, err
}
......
......@@ -26,7 +26,8 @@ import (
func TestFilePerm(t *testing.T) {
log := NewLogger(10000)
log.SetLogger("file", `{"filename":"test.log", "perm": "0600"}`)
// use 0666 as test perm cause the default umask is 022
log.SetLogger("file", `{"filename":"test.log", "perm": "0666"}`)
log.Debug("debug")
log.Informational("info")
log.Notice("notice")
......@@ -39,7 +40,7 @@ func TestFilePerm(t *testing.T) {
if err != nil {
t.Fatal(err)
}
if file.Mode() != 0600 {
if file.Mode() != 0666 {
t.Fatal("unexpected log file permission")
}
os.Remove("test.log")
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment