helpers: Add a comment about file mode for new files
authorBjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com>
Wed, 14 Nov 2018 16:44:04 +0000 (17:44 +0100)
committerBjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com>
Wed, 14 Nov 2018 22:14:51 +0000 (23:14 +0100)
See #5434

helpers/path.go

index f5096e62f36ffed77532656a75d3d3fa6b3f8ced..92b58de8470a7638f9cc5dc7663fd57569a8c81f 100644 (file)
@@ -563,12 +563,13 @@ func OpenFilesForWriting(fs afero.Fs, filenames ...string) (io.WriteCloser, erro
 func OpenFileForWriting(fs afero.Fs, filename string) (afero.File, error) {
        filename = filepath.Clean(filename)
        // Create will truncate if file already exists.
+       // os.Create will create any new files with mode 0666 (before umask).
        f, err := fs.Create(filename)
        if err != nil {
                if !os.IsNotExist(err) {
                        return nil, err
                }
-               if err = fs.MkdirAll(filepath.Dir(filename), 0777); err != nil { // rwx, rw, r before umask
+               if err = fs.MkdirAll(filepath.Dir(filename), 0777); err != nil { //  before umask
                        return nil, err
                }
                f, err = fs.Create(filename)