diff --git a/logs/file.go b/logs/file.go
index beaa9171..42146dae 100644
--- a/logs/file.go
+++ b/logs/file.go
@@ -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
 }
diff --git a/logs/file_test.go b/logs/file_test.go
index 23370947..69a66d84 100644
--- a/logs/file_test.go
+++ b/logs/file_test.go
@@ -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")