aboutsummaryrefslogtreecommitdiffhomepage
path: root/Firebase/Messaging
diff options
context:
space:
mode:
authorGravatar Ryan Wilson <wilsonryan@google.com>2018-05-25 08:55:46 -0400
committerGravatar GitHub <noreply@github.com>2018-05-25 08:55:46 -0400
commit1030d5ec9da07ff8bf0b18beae95558797b26a5b (patch)
treee00f9d983045b22ed3cb225570c78f91ed053156 /Firebase/Messaging
parent8037a4bf79b8ae49162c2b6f099d62ec17a7f283 (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.m4
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;
}