From f19deba809c3d287617b2f040aedf210d6dcb8f4 Mon Sep 17 00:00:00 2001 From: Brian Bland Date: Mon, 28 Dec 2015 15:22:28 -0800 Subject: [PATCH] Relaxes filesystem driver permissions to 0777 (dirs) and 0666 (files) Leaves any further permissions restrictions to the process umask Signed-off-by: Brian Bland --- registry/storage/driver/filesystem/driver.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/registry/storage/driver/filesystem/driver.go b/registry/storage/driver/filesystem/driver.go index 480bd6873..5b495818b 100644 --- a/registry/storage/driver/filesystem/driver.go +++ b/registry/storage/driver/filesystem/driver.go @@ -135,11 +135,11 @@ func (d *driver) WriteStream(ctx context.Context, subPath string, offset int64, fullPath := d.fullPath(subPath) parentDir := path.Dir(fullPath) - if err := os.MkdirAll(parentDir, 0755); err != nil { + if err := os.MkdirAll(parentDir, 0777); err != nil { return 0, err } - fp, err := os.OpenFile(fullPath, os.O_WRONLY|os.O_CREATE, 0644) + fp, err := os.OpenFile(fullPath, os.O_WRONLY|os.O_CREATE, 0666) if err != nil { // TODO(stevvooe): A few missing conditions in storage driver: // 1. What if the path is already a directory?