Skip to content

Commit

Permalink
Merge pull request #7058 from ywk253100/231106_file
Browse files Browse the repository at this point in the history
Truncate the credential file to avoid the change of secret content messing it up
  • Loading branch information
qiuming-best authored Nov 8, 2023
2 parents a674a1e + 6438fc9 commit d139922
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions changelogs/unreleased/7058-ywk253100
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Truncate the credential file to avoid the change of secret content messing it up
2 changes: 1 addition & 1 deletion internal/credentials/file_store.go
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ func (n *namespacedFileStore) Path(selector *corev1api.SecretKeySelector) (strin

keyFilePath := filepath.Join(n.fsRoot, fmt.Sprintf("%s-%s", selector.Name, selector.Key))

file, err := n.fs.OpenFile(keyFilePath, os.O_RDWR|os.O_CREATE, 0644)
file, err := n.fs.OpenFile(keyFilePath, os.O_RDWR|os.O_CREATE|os.O_TRUNC, 0644)
if err != nil {
return "", errors.Wrap(err, "unable to open credentials file for writing")
}
Expand Down

0 comments on commit d139922

Please sign in to comment.