Skip to content

Commit

Permalink
Merge pull request #868 from juliansteenbakker/fix/darwin-query-valid…
Browse files Browse the repository at this point in the history
…ation

fix: remove unsupported query validation
  • Loading branch information
juliansteenbakker authored Jan 26, 2025
2 parents 4d4dc57 + baceecb commit c6e147e
Showing 1 changed file with 0 additions and 10 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -172,16 +172,6 @@ class FlutterSecureStorage {
}

private func validateQueryParameters(params: KeychainQueryParameters) throws {
// Synchronizable checks
if params.isSynchronizable == true {
if params.accessGroup != nil {
throw OSSecError(status: errSecParam, message: "Cannot use kSecAttrSynchronizable with kSecAttrAccessGroup.")
}
// if let itemClass = params.service, !(itemClass == "kSecClassGenericPassword" || itemClass == "kSecClassInternetPassword") {
// throw OSSecError(status: errSecParam, message: "kSecAttrSynchronizable is only supported for passwords.")
// }
}

// Accessibility and access control
if params.accessibilityLevel != nil, params.accessControlSettings != nil {
throw OSSecError(status: errSecParam, message: "Cannot use kSecAttrAccessible and kSecAttrAccessControl together.")
Expand Down

0 comments on commit c6e147e

Please sign in to comment.