Merge pull request #2181 from norio-nomura/write-file-not-apply-permission-to-existing-file

Use `(*os.File).Write()` instead of `os.WriteFile()`

Author: jandubois

pkg/editutil/editutil.go

@@ -63,9 +63,10 @@ func OpenEditor(content []byte, hdr string) ([]byte, error) {
 	}
 	tmpYAMLPath := tmpYAMLFile.Name()
 	defer os.RemoveAll(tmpYAMLPath)
-	if err := os.WriteFile(tmpYAMLPath,
-		append([]byte(hdr), content...),
-		0o600); err != nil {
+	if _, err := tmpYAMLFile.Write(append([]byte(hdr), content...)); err != nil {
+		return nil, err
+	}
+	if err := tmpYAMLFile.Close(); err != nil {
 		return nil, err
 	}
 

pkg/yqutil/yqutil.go

@@ -20,10 +20,13 @@ func EvaluateExpression(expression string, content []byte) ([]byte, error) {
 	}
 	tmpYAMLPath := tmpYAMLFile.Name()
 	defer os.RemoveAll(tmpYAMLPath)
-	err = os.WriteFile(tmpYAMLPath, content, 0o600)
+	_, err = tmpYAMLFile.Write(content)
 	if err != nil {
 		return nil, err
 	}
+	if err = tmpYAMLFile.Close(); err != nil {
+		return nil, err
+	}
 
 	memory := logging.NewMemoryBackend(0)
 	backend := logging.AddModuleLevel(memory)