diff options
author | Ryan Wilson <wilsonryan@google.com> | 2018-05-25 08:55:46 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-05-25 08:55:46 -0400 |
commit | 1030d5ec9da07ff8bf0b18beae95558797b26a5b (patch) | |
tree | e00f9d983045b22ed3cb225570c78f91ed053156 /Firebase/Messaging | |
parent | 8037a4bf79b8ae49162c2b6f099d62ec17a7f283 (diff) |
Temporary fix for Messaging's data collection bit (#1334)
* Temporary fix for Messaging's data collection bit
This ensures that existing behavior still works - InstanceID can sometimes fetch stored values before `FirebaseApp.configure()` is called, causing the global flag to be `NO`. Will file an issue shortly.
* Remove extra whitespaces
Thanks, GitHub editor.
* Fix method being called
Diffstat (limited to 'Firebase/Messaging')
-rw-r--r-- | Firebase/Messaging/FIRMessaging.m | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/Firebase/Messaging/FIRMessaging.m b/Firebase/Messaging/FIRMessaging.m index 972b4fa..3c4d999 100644 --- a/Firebase/Messaging/FIRMessaging.m +++ b/Firebase/Messaging/FIRMessaging.m @@ -173,6 +173,10 @@ NSString *const kFIRMessagingPlistAutoInitEnabled = _loggedMessageIDs = [NSMutableSet set]; _instanceID = instanceID; _messagingUserDefaults = defaults; + + // TODO: Remove this once the race condition with FIRApp configuring and InstanceID + // is fixed. This must be fixed before Core's flag becomes public. + _globalAutomaticDataCollectionEnabled = YES; } return self; } |