Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

File Timestamp Required Reason — Should it be C617.1 instead of 3B52.1 ? #2371

Closed
macdrevx opened this issue Apr 9, 2024 · 1 comment
Closed

Comments

@macdrevx
Copy link

macdrevx commented Apr 9, 2024

Which Version of Lottie are you using?

Lottie 4.4.1

Expected Behavior

Privacy Manifest lists C617.1 as the reason for using file timestamp APIs

Actual Behavior

Privacy Manifest lists 3B52.1 as the reason for using file timestamp APIs

Discussion

Lottie 4.4.1's privacy manifest lists 3B52.1 as the required reason for using file timestamp APIs. Apple defines this reason as:

Declare this reason to access the timestamps, size, or other metadata of files or directories that the user specifically granted access to, such as using a document picker view controller.

This reason doesn't seem to fit with my understanding of Lottie as dealing merely with playback of animation files, so I'm wondering whether it should actually be C617.1:

Declare this reason to access the timestamps, size, or other metadata of files inside the app container, app group container, or the app’s CloudKit container.

At least for our use case, this seems right since we bundle animation files with our app and then play them back with Lottie (without the end-user specifically granting access to those files since they're in the app bundle).

@calda
Copy link
Member

calda commented Apr 15, 2024

Thanks for the suggestion, you're correct. Updated in #2380.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants