aboutsummaryrefslogtreecommitdiffhomepage
path: root/Firebase
diff options
context:
space:
mode:
authorGravatar Ryan Wilson <wilsonryan@google.com>2018-05-25 09:00:03 -0400
committerGravatar GitHub <noreply@github.com>2018-05-25 09:00:03 -0400
commitdd5c3c41b91a8642807303c99ba0c39e56f11e25 (patch)
treec396e0d01e50f56ac49e1b6c659f784e77aca8a3 /Firebase
parent39e7f6515bec8895b2ee56c5003f15c577094b22 (diff)
Temporary fix for Messaging's data collection bit (#1334) (#1336)
* 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')
-rw-r--r--Firebase/Messaging/FIRMessaging.m4
1 files changed, 4 insertions, 0 deletions
diff --git a/Firebase/Messaging/FIRMessaging.m b/Firebase/Messaging/FIRMessaging.m
index d5e0a01..3c1c730 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;
}