couchbase / couchbase-lite-ios

Lightweight, embedded, syncable NoSQL database engine for iOS and MacOS apps.
Apache License 2.0
1.62k stars 297 forks source link

`PrivacyInfo.xcprivacy` file is missing `NSPrivacyCollectedDataTypes` key #3257

Closed Vyazovoy closed 6 months ago

Vyazovoy commented 6 months ago

Describe the bug When I try to generate a privacy report in Xcode the report mentions next:

Screenshot 2024-03-15 at 15 58 50

To Reproduce Try to generate a privacy report

Expected behavior No errors in the report

Additional Info

Looks like it would be better to have all required keys even if corresponding array is empty. Like this

<?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>NSPrivacyTracking</key>
  <false/>
  <key>NSPrivacyCollectedDataTypes</key>
  <array/>
  <key>NSPrivacyTrackingDomains</key>
  <array/>
  <key>NSPrivacyAccessedAPITypes</key>
  <array>
    <dict>
      <key>NSPrivacyAccessedAPIType</key>
      <string>NSPrivacyAccessedAPICategoryFileTimestamp</string>
      <key>NSPrivacyAccessedAPITypeReasons</key>
      <array>
        <string>C617.1</string>
      </array>
    </dict>
  </array>
</dict>
</plist>
pasin commented 6 months ago

Seem like the only two keys are required : NSPrivacyCollectedDataTypes and NSPrivacyAccessedAPITypes. But I doesn't hurt to add all of the keys. Also I noticed that the generated report is empty unless there are something in NSPrivacyTracking and NSPrivacyTrackingDomains.

velicuvlad commented 6 months ago

This has now been ported to the 3.1 line and will be available on the next version. Unfortunately, we don't have a clear timeline for that yet.

manuel-ceschi commented 5 months ago

Hi @velicuvlad . Do you have any news regarding a release date?

velicuvlad commented 5 months ago

Not yet, no. We are aware that this is required by Apple from Spring 2024 onwards and we are actively looking into scheduling a release date to get this in.

velicuvlad commented 4 months ago

3.1.7 has been now released and it includes the fix for this. @manuel-ceschi @Vyazovoy