NordicSemiconductor / IOS-DFU-Library

OTA DFU Library for Mac and iOS, compatible with nRF5x SoCs

Home Page:http://www.nordicsemi.com

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Privacy Manifest

mateusforgi opened this issue · comments

Information
Apple has announced the Privacy Manifest, required by apps and SDKs.

Third-party SDKs need to provide their own privacy manifest files that record the types of data they collect. Your app’s privacy manifest file doesn’t need to cover data collected by third-party SDKs that your app links to.

We develop a SDK that has the iOS-DFU-Library as its dependencies, we are already providing the Privacy Manifest for our SDK, however for the future iOS-DFU-Library needs to also have its own file.

Your question
When are you guys planning to include it on the iOS-DFU-Library?

@mateusforgi we will do that, thanks for the reminder. Sometimes we receive a wave of things we need to check, fix or look at immediately and things slip. Sorry about that, we'll look into that on Monday.

@dinesharjani Thanks. We have some time until it becomes mandatory.

According to apple:
From Fall 2023 you’ll receive an email from Apple if you upload an app to App Store Connect that uses required reason API without describing the reason in its privacy manifest file. From Spring 2024, apps that don’t describe their use of required reason API in their privacy manifest file won’t be accepted by App Store Connect.

@mateusforgi I made this PR. Could you have a look? #513

Hi, I went though the list of all privacy usages listed here: https://developer.apple.com/documentation/bundleresources/privacy_manifest_files/describing_data_use_in_privacy_manifests and the restricted API listed here: https://developer.apple.com/documentation/bundleresources/privacy_manifest_files/describing_use_of_required_reason_api and I find nothing that we're using in the DFU library.

I found several uses of the API in ZipFoundation library, which the DFU library is using under the hood, e.g.: https://github.com/search?q=repo%3Aweichsel%2FZIPFoundation%20modificationDate&type=code
but those are not exposed and not used by the DFU library.

In my opinion we can just use 3B52.1 for File Timestamp APIs, to be on the safe side.

@dinesharjani looks good to me. Just one think I am not sure, If we can remove NSPrivacyTrackingDomains key. I assume we can, but the doc does not say anything explicitly about it, it only mention to leave it empty.

If you set NSPrivacyTracking to true then you need to provide at least one internet domain in NSPrivacyTrackingDomains; otherwise, **you can provide zero or more domains**.

I will create an issue on ZipFoundation also.
Thanks for acting so quickly!

@dinesharjani @philips77 ZIPFoundation already made the changes and put in on the develop branch, see thread here. However they are using it as a resource file, however on my tests Xcode is not generating the privacy report combining the manifests of the SDKs, it only takes into account the main app manifest. I am following the threads on Firebase library and Facebook to see how they will handle it.

@mateusforgi appreciate it. If you could keep us updated, that'd be great.

@dinesharjani @philips77 I confirmed. We need to add the privacy report as resource file, like ZIPFoundation is doing here.

Thanks, we'll add it.