Privacy Manifest support would be needed in realm-wrappers about realm-dotnet HOT 10 CLOSED

Thanks for bringing it to our attention. We'll schedule some work to make sure it's done before May 1st.

from realm-dotnet.

It seems that our usage is already covered by the default privacy manifest that MAUI needs, so we should be fine. I've also tried to send a test app to Apple for review and did not get any warning/errors about the privacy manifest, so hopefully we are already covered.

That said, feel free to reopen this if you get any errors about Realm when submitting to Apple.

from realm-dotnet.

Thanks a lot for your comment. I'm relieved to hear that.

By the way, I previously wrote that the declaration in the manifest can be appropriated as is. But after that, I found that realm-wrappers seems to use not only fstat and stat, but also statvfs.
realm-swift's PrivacyInfo.xcprivacy only declares NSPrivacyAccessedAPICategoryFileTimestamp (for fstat and stat).
In realm-wrappers, it might be also needed to declare NSPrivacyAccessedAPICategoryDiskSpace in the manifest.

<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
<dict>
	<key>NSPrivacyAccessedAPITypes</key>
	<array>
		<dict>
			<key>NSPrivacyAccessedAPIType</key>
			<string>NSPrivacyAccessedAPICategoryFileTimestamp</string>
			<key>NSPrivacyAccessedAPITypeReasons</key>
			<array>
				<string>C617.1</string>
			</array>
		</dict>
		<dict>
			<key>NSPrivacyAccessedAPIType</key>
			<string>NSPrivacyAccessedAPICategoryDiskSpace</string>
			<key>NSPrivacyAccessedAPITypeReasons</key>
			<array>
				<string>E174.1</string>
			</array>
		</dict>
	</array>
	<key>NSPrivacyTracking</key>
	<false/>
	<key>NSPrivacyTrackingDomains</key>
	<array>
	</array>
	<key>NSPrivacyCollectedDataTypes</key>
	<array>
	</array>
</dict>
</plist>

from realm-dotnet.

I thought that the call to statvfs was only hit in testing code, but after double-checking it's also called when backing up realm files prior to file format upgrades, so RealmSwift needs to add that too. Thanks for catching that.

from realm-dotnet.

Looking at the documentation, it seems that the minimum privacy manifest necessary for iOS application already contains the keys that we wanted to include in out privacy manifest. For this reason we believe we should not do anything on our side, so I'm closing this ticket but feel free to reopen it if you experience any issue.

On a side note, @takeshik thanks a lot for reporting this and also spending the time to investigate how to fix it. I verified that your patch puts the privacy framework in the right place for iOS before noticing we probably don't need it. So thanks a lot on our side 😄

from realm-dotnet.

➤ PM Bot commented:

Jira ticket: RNET-1119

from realm-dotnet.

Little bit of confusing info but here are two links from Microsoft about this:

• https://learn.microsoft.com/en-us/dotnet/maui/ios/privacy-manifest?view=net-maui-8.0#create-a-privacy-manifest
• https://devblogs.microsoft.com/dotnet/apple-privacy-manifest-support/

from realm-dotnet.

Note that it appears we don't have to add this by May 1st as the Realm .NET SDK is not included in that list: https://developer.apple.com/support/third-party-SDK-requirements/.

from realm-dotnet.

Yes, fortunately we're not there. It's mostly a nice thing so developers don't have to do it themselves

from realm-dotnet.

Hello @papafe

i'm wondering if Realm dotnet uses these C# .Net framework APIs

if so, it would be needed to include a privacy manifest file into realm-wrappers.framework as @takeshik mentioned

i might be wrong, it's quite confusing :(

from realm-dotnet.