Alternative approach

Sources/Storage/NetworkDriver.swift

@@ -5,11 +5,20 @@ import Foundation
 public protocol NetworkDriver: Service {
     var pathBuilder: PathBuilder { get set }
 
-    func upload(entity: inout FileEntity, access: AccessControlList, on container: Container) throws -> Future<String>
+    func upload(entity: inout FileEntity, access: AccessControlList?, on container: Container) throws -> Future<String>
     func get(path: String, on container: Container) throws -> Future<Response>
     func delete(path: String, on container: Container) throws -> Future<Response>
 }
 
+extension NetworkDriver {
+    func upload(
+        entity: inout FileEntity,
+        on container: Container
+    ) throws -> Future<String> {
+        try upload(entity: &entity, access: nil, on: container)
+    }
+}
+
 public final class S3Driver: NetworkDriver {
     public enum Error: Swift.Error {
         case nilFileUpload
@@ -59,7 +68,7 @@ public final class S3Driver: NetworkDriver {
 
     public func upload(
         entity: inout FileEntity,
-        access: AccessControlList = .publicRead,
+        access: AccessControlList?,
         on container: Container
     ) throws -> Future<String> {
         guard let bytes = entity.bytes else {
@@ -92,7 +101,7 @@ public final class S3Driver: NetworkDriver {
             bytes: Data(bytes),
             path: path,
             contentType: mime,
-            access: access,
+            access: access ?? .publicRead,
             on: container
         ).transform(to: path)
     }

Sources/Storage/Storage.swift

@@ -25,7 +25,7 @@ public class Storage {
     @discardableResult
     public static func upload(
         entity: inout FileEntity,
-        access: AccessControlList = .publicRead,
+        access: AccessControlList? = nil,
         on container: Container
     ) throws -> Future<String> {
         let networkDriver = try container.make(NetworkDriver.self)
@@ -51,7 +51,7 @@ public class Storage {
         fileExtension: String? = nil,
         mime: String? = nil,
         folder: String? = nil,
-        access: AccessControlList = .publicRead,
+        access: AccessControlList? = nil,
         on container: Container
     ) throws -> Future<String> {
         var entity = FileEntity(
@@ -62,7 +62,7 @@ public class Storage {
             mime: mime
         )
 
-        return try upload(entity: &entity, on: container)
+        return try upload(entity: &entity, access: access, on: container)
     }
 
     /**
@@ -82,7 +82,7 @@ public class Storage {
         fileName: String? = nil,
         fileExtension: String? = nil,
         folder: String? = nil,
-        access: AccessControlList = .publicRead,
+        access: AccessControlList? = nil,
         on container: Container
     ) throws -> Future<String> {
         let (bytes, type) = try dataURI.dataURIDecoded()
@@ -92,6 +92,7 @@ public class Storage {
             fileExtension: fileExtension,
             mime: type,
             folder: folder,
+            access: access,
             on: container
         )
     }