realm / realm-core / 2207

91%

Build Type

push

Evergreen

Committed by web-flow

Commit Message

RCORE-2070 Allow setting a security access group for the metadata realm keychain (#7552)

Access groups are shared storage for one or more apps on iOS (and other Apple
platforms). Sharing a metadata Realm between apps requires placing the file in
the access group storage and storing the encryption key in the access group's
keychain.

Including the bundle ID in the service name breaks sharing the key between
apps, as different apps will have different bundle IDs. For everything but
un-sandboxed macOS there wasn't actually any reason to include the bundle ID in
the first place, as each app has its own keychain anyway. As such, this
switches back to not including it. On macOS this continues to include the
bundle ID when not using an access group, as otherwise different applications
could conflict with each other. This means that sharing users between macOS
applications will currently only work if an encryption key is explicitly set or
if the applications have sandboxing enabled.

Since this is slightly changing how keys are stored anyway, it also switches to
using unique keys per server app ID rather than always using "metadata" as the
account name.

The keychain code was mostly multiprocess-safe, but there was one race
condition when two apps generated a new key at once which is fixed.

Run Details

96124 of 180408 branches covered (53.28%)

181 of 197 new or added lines in 10 files covered. (91.88%)

2 existing lines in 2 files now uncovered.

245163 of 266226 relevant lines covered (92.09%)

39533103.77 hits per line