-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Merge IID removal code to master #7814
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
Changes from 30 commits
8ec05d6
8ba1a1c
3d8e2c7
012bc14
1561602
42ac674
141b582
eda686b
dd085db
cea68ec
ded8291
0582395
8356e85
3090073
e1fbde9
482d039
5df1836
41877bd
4f33b0e
30335a3
162e754
2471cd8
0092172
f8fdf7f
bb1230c
a379ce2
3f18619
f74aef6
55579f1
7570b14
3293bd7
f8ca45f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -42,7 +42,7 @@ device, and it is completely free. | |
| 'Firebase/InstanceID/Public/*.h', | ||
| 'FirebaseInstallations/Source/Library/Private/*.h', | ||
| ] | ||
| s.requires_arc = base_dir + 'Sources/*.m' | ||
| s.requires_arc = base_dir + 'Sources/**/*.m' | ||
| s.public_header_files = base_dir + 'Sources/Public/FirebaseMessaging/*.h' | ||
| s.library = 'sqlite3' | ||
| s.pod_target_xcconfig = { | ||
|
|
@@ -54,8 +54,9 @@ device, and it is completely free. | |
| s.tvos.framework = 'SystemConfiguration' | ||
| s.osx.framework = 'SystemConfiguration' | ||
| s.weak_framework = 'UserNotifications' | ||
| s.dependency 'FirebaseCore', '~> 7.0' | ||
| s.dependency 'FirebaseInstanceID', '~> 7.0' | ||
| s.dependency 'FirebaseInstallations', '~> 7.0' | ||
| s.dependency 'FirebaseCore', '~> 7.0' | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. nit: alphabetize dependencies There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Done. |
||
| s.dependency 'GoogleUtilities/AppDelegateSwizzler', '~> 7.0' | ||
| s.dependency 'GoogleUtilities/Reachability', '~> 7.0' | ||
| s.dependency 'GoogleUtilities/Environment', '~> 7.0' | ||
|
|
@@ -68,7 +69,10 @@ device, and it is completely free. | |
| :osx => osx_deployment_target, | ||
| :tvos => tvos_deployment_target | ||
| } | ||
| unit_tests.source_files = 'FirebaseMessaging/Tests/UnitTests*/*.{m,h,swift}' | ||
| unit_tests.source_files = [ | ||
| 'FirebaseMessaging/Tests/UnitTests*/*.{m,h,swift}', | ||
| 'SharedTestUtilities/URLSession/*.[mh]', | ||
| ] | ||
| unit_tests.requires_app_host = true | ||
| unit_tests.pod_target_xcconfig = { | ||
| 'CLANG_ENABLE_OBJC_WEAK' => 'YES' | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,6 @@ | ||
| #unreleased | ||
|
||
| - [changed] Refactor Messaging to internally not depending on InstanceID, but can co-exist. Will remove InstanceID dependency in the next Firebase breaking change. (#7814) | ||
|
|
||
| # 2021-02 -- v7.7.0 | ||
| - [fixed] Fixed an issue in which, when checking storage size before writing to disk, the client was checking document folders that were no longer used. (#7480) | ||
|
|
||
|
|
||
Large diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -31,9 +31,52 @@ FOUNDATION_EXPORT NSString *const kFIRMessagingMessageLinkKey; | |
| FOUNDATION_EXPORT NSString *const kFIRMessagingRemoteNotificationsProxyEnabledInfoPlistKey; | ||
| FOUNDATION_EXPORT NSString *const kFIRMessagingSubDirectoryName; | ||
|
|
||
| // Notifications | ||
| #pragma mark - Notifications | ||
|
|
||
| FOUNDATION_EXPORT NSString *const kFIRMessagingCheckinFetchedNotification; | ||
| FOUNDATION_EXPORT NSString *const kFIRMessagingAPNSTokenNotification; | ||
| FOUNDATION_EXPORT NSString *const kFIRMessagingDefaultGCMTokenFailNotification; | ||
| FOUNDATION_EXPORT NSString *const kFIRMessagingRegistrationTokenRefreshNotification; | ||
|
|
||
| FOUNDATION_EXPORT const int kFIRMessagingSendTtlDefault; // 24 hours | ||
|
|
||
| /** | ||
| * Value included in a structured response or GCM message from IID, indicating | ||
|
||
| * an identity reset. | ||
| */ | ||
| FOUNDATION_EXPORT NSString *const kFIRMessaging_CMD_RST; | ||
|
|
||
| #pragma mark - Miscellaneous | ||
|
|
||
| /// The scope used to save the IID "*" scope token. This is used for saving the | ||
| /// IID auth token that we receive from the server. This feature was never | ||
| /// implemented on the server side. | ||
| FOUNDATION_EXPORT NSString *const kFIRMessagingAllScopeIdentifier; | ||
| /// The scope used to save the IID "*" scope token. | ||
| FOUNDATION_EXPORT NSString *const kFIRMessagingDefaultTokenScope; | ||
|
|
||
| /// Subdirectory in search path directory to store InstanceID preferences. | ||
| FOUNDATION_EXPORT NSString *const kFIRMessagingInstanceIDSubDirectoryName; | ||
|
|
||
| /// The key for APNS token in options dictionary. | ||
| FOUNDATION_EXPORT NSString *const kFIRMessagingTokenOptionsAPNSKey; | ||
|
|
||
| /// The key for APNS token environment type in options dictionary. | ||
| FOUNDATION_EXPORT NSString *const kFIRMessagingTokenOptionsAPNSIsSandboxKey; | ||
|
|
||
| /// The key for GMP AppID sent in registration requests. | ||
| FOUNDATION_EXPORT NSString *const kFIRMessagingTokenOptionsFirebaseAppIDKey; | ||
|
|
||
| FOUNDATION_EXPORT NSString *const kFIRMessagingAPNSTokenType; | ||
|
|
||
| /// The key to enable auto-register by swizzling AppDelegate's methods. | ||
| FOUNDATION_EXPORT NSString *const kFIRMessagingAppDelegateProxyEnabledInfoPlistKey; | ||
|
|
||
| /// Error code for missing entitlements in Keychain. iOS Keychain error | ||
| /// https://forums.developer.apple.com/thread/4743 | ||
| FOUNDATION_EXPORT const int kFIRMessagingSecMissingEntitlementErrorCode; | ||
|
|
||
| /// The key for InstallationID or InstanceID in token request. | ||
| FOUNDATION_EXPORT NSString *const kFIRMessagingParamInstanceID; | ||
|
|
||
| #endif | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now that protobuf's non-arc files are gone, this line could be deleted since requires_arc is the default
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done