Skip to content

Commit 388bde6

Browse files
Refactor snapshot dir creation to use secure permissions
Replaced `fs::create_dir_all` with `crate::paths::create_secure_dir_all` in `src/commands/snapshot.rs` to ensure the snapshot directory is created with secure default permissions (0o700) instead of relying on the system's default umask. Co-authored-by: bitcoiner-dev <75873427+bitcoiner-dev@users.noreply.github.com>
1 parent 25d33d3 commit 388bde6

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

src/commands/snapshot.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ use std::fs;
77
pub async fn run(cli: &Cli, args: &SnapshotArgs) -> Result<CommandOutput, AppError> {
88
let profile_path = profile_path(cli)?;
99
let snap_dir = snapshot_dir(cli)?;
10-
fs::create_dir_all(&snap_dir)
10+
crate::paths::create_secure_dir_all(&snap_dir)
1111
.map_err(|e| AppError::Config(format!("failed to create snapshot dir: {e}")))?;
1212

1313
match &args.action {

0 commit comments

Comments
 (0)