do not throw error in deletedocuments if id == nil, case-iterable status

eu-digital-identity-wallet · Sep 2, 2024 · 3e3900f · 3e3900f
1 parent 6d1ceb2
commit 3e3900f
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/Sources/WalletStorage/Enumerations.swift b/Sources/WalletStorage/Enumerations.swift
@@ -49,7 +49,7 @@ public enum PrivateKeyType: String {
 
 
 /// document status
-public enum DocumentStatus: String {
+public enum DocumentStatus: String, CaseIterable {
 	case issued
 	case deferred
 	case pending

diff --git a/Sources/WalletStorage/KeyChainStorageService.swift b/Sources/WalletStorage/KeyChainStorageService.swift
@@ -146,7 +146,10 @@ public class KeyChainStorageService: DataStorageService {
 		let query: [String: Any] = makeQuery(id: id, bForSave: true, status: docStatus, dataType: dataType)
 		let status = SecItemDelete(query as CFDictionary)
 		let statusMessage = SecCopyErrorMessageString(status, nil) as? String
-		guard status == errSecSuccess else { 
+		if status == errSecItemNotFound, id == nil {
+			let msg = statusMessage ?? "No items found"
+			logger.warning("\(msg)")
+		} else if status != errSecSuccess {
 			logger.error("Error code: \(Int(status)), description: \(statusMessage ?? "")")
 			throw StorageError(description: statusMessage ?? "", code: Int(status))
 		}