-
Notifications
You must be signed in to change notification settings - Fork 24.5k
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
Priavacy Policy aggregation errors on empty dict #44401
Comments
I've added a reproducer |
@aleqsio @cipolleschi can we look at this as this is a newly introduced feature? |
This patch works for me: diff --git a/node_modules/react-native/scripts/cocoapods/privacy_manifest_utils.rb b/node_modules/react-native/scripts/cocoapods/privacy_manifest_utils.rb
index 7600829..f5fd6d5 100644
--- a/node_modules/react-native/scripts/cocoapods/privacy_manifest_utils.rb
+++ b/node_modules/react-native/scripts/cocoapods/privacy_manifest_utils.rb
@@ -111,6 +111,7 @@ module PrivacyManifestUtils
accessed_api_types.each do |accessed_api|
api_type = accessed_api["NSPrivacyAccessedAPIType"]
reasons = accessed_api["NSPrivacyAccessedAPITypeReasons"]
+ next if api_type.nil? || reasons.nil?
used_apis[api_type] ||= []
used_apis[api_type] += reasons
end |
# Why fix https://expo.dev/accounts/expo-ci/projects/updates-e2e/builds/b9722d1c-b4d6-4d0a-aaaa-09e519875b9c # How - temporarily apply fix for facebook/react-native#44401 - suppress expo-dev-client lookup warning
I have this error when migrated from 0.73.8 to 0.74.1
|
I have created ticket with patch-file for mine issue: #44437 |
Same issue react-native 0.74.1 Generating Pods project no implicit conversion of nil into Array /Users/rodrigodiasdefigueiredo/Desktop/parksharing/node_modules/react-native/scripts/cocoapods/privacy_manifest_utils.rb:115:in |
You can turn off the privacy manifest aggregation by adding The fix for your issue should land in the next patch release. |
something like this?
|
duplicated of #44400 |
Turning off
|
New version of 0.74.2 is out and it should have all the fixes to the Privacy manifest we need! Thank you everyone for the patience! |
I've updated to 0.74.2 and unfortunately still seeing:
|
@david-cahill could you add the rest of the stacktrace, please? |
@cipolleschi still seeing the same error with RN 0.74.2 (upgrading from 0.73.8)
PrivacyInfo.xcprivacy:
|
Also getting the exact same error "undefined method `path' for nil:NilClass" on pod install. Not sure if it's related, but the project is a monorepo with the RN app sitting inside a workspace beneath root
|
we recently merged a change that should fix it: 1bec961. Thanks for your patience with this. Meanwhile, you can try to apply the above commit as |
@cipolleschi unfortunately it appears that 0.74.3 does not contain the fix. Suppose we'll need to wait for 0.74.4 |
Sorry for that, I'll make sure this is picked in 0.74.4 |
Description
pod update will failed when it tries to aggregate a privacy policy with an empty dict. For example
I came across this in https://github.com/TimOliver/TOCropViewController which is a dependency of https://github.com/ivpusic/react-native-image-crop-picker
Steps to reproduce
React Native Version
0.74.1
Affected Platforms
Build - MacOS
Output of
npx react-native info
Stacktrace or Logs
Reproducer
https://github.com/johnf/rn-priacy-policy-aggregation-bug
Screenshots and Videos
No response
The text was updated successfully, but these errors were encountered: