Skip to content

Commit e3b893e

Browse files
brennanMKEroyjit
authored andcommitted
chore(storage): updates logging for Storage delegate functions (#97)
1 parent 0814be0 commit e3b893e

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

AmplifyPlugins/Storage/AWSS3StoragePlugin/Support/Internal/StorageServiceSessionDelegate.swift

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -107,12 +107,13 @@ extension StorageServiceSessionDelegate: URLSessionTaskDelegate {
107107
switch transferTask.transferType {
108108
case .multiPartUploadPart(let uploadId, let partNumber):
109109
guard let multipartUploadSession = storageService.findMultipartUploadSession(uploadId: uploadId) else {
110-
logger.info("MultipartUpload not found for uploadId: \(uploadId)")
110+
logURLSessionActivity("MultipartUpload not found for uploadId: \(uploadId)")
111111
return
112112
}
113113

114114
guard let eTag = task.eTag else {
115-
logger.error("Completed upload part does not include header value for ETAG: [\(partNumber), \(uploadId)]")
115+
let message = "Completed upload part does not include header value for ETAG: [\(partNumber), \(uploadId)]"
116+
logURLSessionActivity(message, warning: true)
116117
multipartUploadSession.handle(uploadPartEvent: .failed(partNumber: partNumber, error: StorageError.unknown("Upload for part number does not include value for eTag", nil)))
117118
return
118119
}
@@ -141,7 +142,7 @@ extension StorageServiceSessionDelegate: URLSessionTaskDelegate {
141142
switch transferTask.transferType {
142143
case .multiPartUploadPart(let uploadId, let partNumber):
143144
guard let multipartUploadSession = storageService.findMultipartUploadSession(uploadId: uploadId) else {
144-
logger.info("MultipartUpload not found for uploadId: \(uploadId)")
145+
logURLSessionActivity("MultipartUpload not found for uploadId: \(uploadId)")
145146
return
146147
}
147148

0 commit comments

Comments
 (0)